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
mgallien |
Elisa |
Design changes for the sidebar according to the mockups in the ML
No Linters Available |
No Unit Test Coverage |
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.
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.
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