Design changes for the sidebar according to the mockups in the ML
In this patch
- Make sidebar smaller
- Make the selected view use the highlight color for the text and icon
- Add a small margin to align with the views titles
- Remove backgrounds
Details
Details
- Reviewers
mgallien - Group Reviewers
Elisa - Commits
- R255:3f9782721913: Sidebar design changes
Diff Detail
Diff Detail
- Repository
- R255 Elisa
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
No line between the sidebar and the main view? Not sure that's a good idea. Separating dissimilar elements helps guide the eye to different parts of the app. Without enough separation, parts that are different can start to blend together.
Comment Actions
I tend to agree. If we want to make the sidebar looks lighter, we can still add a very thin line to separate it from the main view.
Comment Actions
TBH I don't think the sidebar and view are too separate, but the line does help separate things better (specially when scrolling). I duplicated the playlist separator line for this. I've also made the sidebar a bit smaller to slide another album per row for the album view