gateship-one / odyssey

Odyssey music player
GNU General Public License v3.0
232 stars 39 forks source link

Allow customizing lists #117

Open DragoonAethis opened 6 years ago

DragoonAethis commented 6 years ago

Currently most lists in Odyssey are very generous when it comes to row height, which is mostly okay, but some users (like yours truly) prefer a bit more cramped layout. The ability to use compact lists results in more entries being displayed on the screen at the same time - at the moment Odyssey can show up to 9 entries on my phone (1080p screen with DPI set to 400), whereas GoneMAD can display 16.

I could give this a try if you'd accept such a feature (extra toggle in settings, "Use compact lists" or something along these lines)?

djselbeck commented 6 years ago

I would accept list item height down to 48dp as an option.

Everything below 48dp is against the design guidelines and would present a usability challenge.

Size elements at least 48dp high and wide to ensure a physical size of about 9mm regardless of screen size.

From: https://material.io/guidelines/layout/metrics-keylines.html#metrics-keylines-touch-target-size

The option should be an additional choice between the existing grid and listview preference.