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
Update look of boxes for consistency (ErrorHandler, Updater, ...) #8199
Conversation
Want to see the screenshot tests before making any decision.
Why did you remove the radius? It doesn't look really good IMO. Edit: ok now I've seen all the prs merged in the night |
box-shadow: 0 1px 2px 0 #ccc; | ||
-webkit-box-shadow: 0 1px 6px 0 #ccc; | ||
-moz-box-shadow: 0 1px 6px 0 #ccc; | ||
box-shadow: 0 1px 6px 0 #ccc; |
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.
No need for the aliases.
as described I changed it for consistency to reuse the card design there as well. I kinda like it better since it looks like this as well in Dashboard, Visitor Log, Sites Manager, Marketplace (kinda), ... and we use border-radius less and less often. I updated the links to the diffs |
yes, as I said one needs to be aware of the pr created and merged during the night to understand where this comes from. |
We're making improvements on merging PRs faster :) |
@mnapoli removing borders is a bit of an experiment in simplifying further and making style consistent... it was not so clear whether to do it or not (or with a 1 or 2px border radius), but I'm keen to try without and see where it leads I think this PR is the last Design change for Piwik 2.14.0 - we did really well at refreshing the design in this short time 👍 our next big step in terms of improving the design will be the Left Menu #7589 (and maybe review the top menu too). |
Update look of boxes for consistency (ErrorHandler, Updater, ...)
We removed the border-radius and another border from the widgets in dashboard and also started using the card design in some places so thought might be consistent to use no border there as well
UI tests: http://builds-artifacts.piwik.org/ui-tests.no_border_radius_in_boxes/13715.7/
eg
http://builds-artifacts.piwik.org/ui-tests.no_border_radius_in_boxes/13715.7/CoreUpdaterDb_updated
http://builds-artifacts.piwik.org/ui-tests.no_border_radius_in_boxes/13715.7/UIIntegrationTest_db_connect_error
http://builds-artifacts.piwik.org/ui-tests.no_border_radius_in_boxes/13715.7/UIIntegrationTest_widgetize_apidisallowed
FYI: I noticed there is the background-color #f0f0f0 used but in Piwik itself we often use #f2f2f2. I didn't change it as I thought this color might be used in the future instead