TeamNewPipe/NewPipe

Increase tablet mode breakpoint

Open

#6.067 aberto em 13 de abr. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Java (3.577 forks)batch import
GUIfeature requestgood first issue

Métricas do repositório

Stars
 (38.153 stars)
Métricas de merge de PR
 (Mesclagem média 9d 17h) (17 fundiu PRs em 30d)

Description

Checklist

Describe the feature you want

Currently, NewPipe determines whether tabled mode should be used based whether the smallest width is <480dp or >=480dp.

According to the the Android developer guide (https://developer.android.com/training/multiscreen/screensizes) the breakpoint should be at 600dp. They have the following quantifiers there:

  • 320dp: a typical phone screen (240x320 ldpi, 320x480 mdpi, 480x800 hdpi, etc).
  • 480dp: a large phone screen ~5" (480x800 mdpi).
  • 600dp: a 7” tablet (600x1024 mdpi).
  • 720dp: a 10” tablet (720x1280 mdpi, 800x1280 mdpi, etc).

This is also consistent with e.g. the Android System UI, which switches into tablet mode at a smallest width of >=600dp.

Is your feature request related to a problem? Please describe it

I run my phone with a smallest width of ~500dp. This defaults NewPipe into tablet mode. PR #6059 fixes the problem for me, but I think the automatic tablet mode threshold is not where it should be.

How will you/everyone benefit from this feature?

This is a really small fix (literally just change a number) and causes phones with larger screens to be detected as phones. I am pretty sure there is no tablet out there with a (default) smallest width of <600dp.

Guia do colaborador