-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UX fixups followup to SPUR session with the team #477
Conversation
b2a5b2a
to
059508c
Compare
ece36b9
to
0d77bea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently working on building the VM. I'm looking at the diff right now and it looks good. I have a comment with a suggestion... what do you think?
501b2f4
to
2c7d0d3
Compare
Patternfly configures the text with grey color.
…nt point Like in the mockups.
What we are trying to achieve with this is to make it clearer that the user will perform storage confoguration outside of anaconda with that option.
2c7d0d3
to
295d2f0
Compare
I've added a disk to a test VM and it has mount point mapping with a bold string.
(Most options shouldn't be shown if they cannot be done.) For example, here's the example mockup where there's just 1 empty disk with no partitions: |
We should remove "Operating system" and the space between it and Language, as we did run into some problems with verticality (especially with a complex storage and/or in small screen sizes, like VMs) and also since it mainly redundantly repeats what's in the header anyway. Additionally, there's too much space to the left of the disk names (vda2, vda3) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with what's here, and we can iterate with more changes on top. It's already a definite improvement. Thanks!
No description provided.