GNOME Bugzilla – Bug 691578
On application hover: corners are not rounded enough on application view in the dash.
Last modified: 2013-08-26 18:59:45 UTC
Dodgy: http://i50.tinypic.com/2z6fj0g.png Fine: http://i47.tinypic.com/ziunpc.png I believe you can see what the difference is in those two screenshots - the highlight when hovered over an application is two large, or its corners are too sharp on the 1st one, and the devs have probrably just used that same image for both of them. This is not a software bug!
I don't see the difference between the two photos, but I agree that having more rounded corners would look better.
Good catch - the border radius doesn't match the radius of the running indicator in the app view. Looking at the css, it seems that we have a single style class for all the app icons. I suspect that we'll need a separate style class for the app icons in the app view as opposed to the dash.
(In reply to comment #2) > Good catch - the border radius doesn't match the radius of the running > indicator in the app view. > > Looking at the css, it seems that we have a single style class for all the app > icons. I suspect that we'll need a separate style class for the app icons in > the app view as opposed to the dash. Hm, I was just going to submit this to every detail matters but it appears it has caught your attention already. Do you know if a dev is working on this?
I am interested to work on this bug
Created attachment 237997 [details] [review] Added border radius to application icons in the grid on hover
Created attachment 238026 [details] [review] Added border radius for icons in application grid and not in dash
Can you squash these two fixes together?
Created attachment 238027 [details] [review] Added border radius for icons in application grid and not in dash
Created attachment 238286 [details] [review] updated patch This looks good to me. Thanks! I had to respin the patch because it didn't apply.
Thanks for the fix, Bharath!
Created attachment 238940 [details] [review] theme: Adjust border-radius for focus indication to match hover Commit 7b3a689aadd changed the border-radius of the hover effect to match the prelight effect, but not the focus indication, so the focused item now changes corners on hover. Fix this by using a consistent border-radius.
Review of attachment 238940 [details] [review]: Can we just make the border-radius always exist instead of special-casing hover and focus?
Attachment 238940 [details] pushed as 603f528 - theme: Adjust border-radius for focus indication to match hover (In reply to comment #12) > Can we just make the border-radius always exist instead of special-casing hover > and focus? Sure ...
It seems that the app search results still have the old, smaller border-radius.(Which means it's now different between app search and app picker too.)
Created attachment 253166 [details] [review] same border radius for app picker and search result icons This patch gives the old 4px border-radius to the dash icons, new 10px to app search results and app picker icons and 5px to the search-provider-icon and the list-search-results.