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
fix: button consistency in logs panel #9945 #10016
fix: button consistency in logs panel #9945 #10016
Conversation
…us event > change the position of fa-check icon in button - when active Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
…t issue Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Codecov ReportBase: 45.76% // Head: 45.76% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #10016 +/- ##
=======================================
Coverage 45.76% 45.76%
=======================================
Files 236 236
Lines 28529 28529
=======================================
Hits 13055 13055
Misses 13669 13669
Partials 1805 1805 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Thanks, @Marvin9! What if, instead of darkening the "active" buttons and adding a small green checkmark, we just added a proper square checkbox to the left of each icon (inside the button) which will become checked when activated. That would allow these boxes to remain "bright" (not confusing them with the darker "action" buttons) and very clearly mark them as either active or inactive. |
Thanks for the suggestion @crenshaw-dev. I will change the design |
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
0976e74
to
a9dfe32
Compare
With new changes Screen.Recording.2022-07-24.at.3.44.01.PM.mov |
I like it! Will wait for UI folks to weigh in. :-) |
Thanks @crenshaw-dev. Can you please mention here who should review and approve? |
Added @rbreeze as a review. Remington, feel free to punt to another UI-capable person. :-) |
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 think the consistency improvements are great, and agree that each button should have consistent behavior and a fixed width. This feels like a big improvement already, but I see 2 different options that could make it look even better:
- All buttons, even the ones with checkboxes, should be the same dark color
OR
- We remove the checkbox paradigm from all buttons. Buttons that can be enabled / disabled would be hollow when disabled and filled in when enabled.
Option 1 is more explicit and a smaller change to the current state of this PR. Personally I think Option 2 would look cleaner, but may be more confusing. @alexec or @alexmt do you have an opinion here?
Thanks for the suggestion @rbreeze. The reason to avoid dark colour for toggles is that it confuses with static dark buttons (eg. copy, download). As soon as both groups have same color (darker one), they seem to loose the context, unless they have checkbox. Let me know what you think on it. |
Fwiw, I agree that option 2 is slightly better, but I'm happy with the current improvement over the previous state. |
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Fixes #9945
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist: