Reduce the size of thumbnails on big screens to fit more grid columns #9858
Conversation
Reverts part of TeamNewPipe#9310, which introduced bigger grid thumbnail sizes on big screens, because some users reported not being happy about having too few grid columns. See TeamNewPipe#9310 (comment) .
|
Kudos, SonarCloud Quality Gate passed!
|
|
Its better, but why not go back to 5 or 6 grids, or at least give that choice? And if possible even removing a lot of the white space/border space between thumbnails, this from 3 to 4 is just mimicking mobile youtube, currently on newpipe it shows me a 3 grid and on m.youtube shows me a 4 grid but on m.youtube the 4 thumbnails seem even bigger than the 3 on newpipe just because there is is very little border space between the thumbnails, you guys have to give us better usability or else whats the point? |
|
4 grids is the perfect size, please proceed. 🤔 😀 |
|
If in 0.24.1 you had 6 items per row, with this PR you would probably get 5. |
|
Thank you for testing. I know this is not the optimal solution, but at least it will hopefully suit a bit better both people who want large thumbnails, and people who want more data on the screen at once. I'm merging this and soon we'll ship 0.25.1. |
|
not happy, but i still appreciate your work, thanks! |
What is it?
Description of the changes in your PR
Reverts a small piece of #9310, which introduced bigger grid thumbnail sizes on big screens, because some users reported not being happy about having too few grid columns. See #9310 (comment).
Before/After Screenshots/Screen Record
Fixes the following issue(s)
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.
Due diligence