Speedmenu layout in landscape mode

[defini-tiv]

Checklist

• I checked, but didn't find any duplicates (open OR closed) of this issue in the repo.
• I have read the contribution guidelines given at https://github.com/TeamNewPipe/NewPipe/blob/HEAD/.github/CONTRIBUTING.md.
• This issue contains only one feature request. I will open one issue for every feature I want to request.

Describe the feature you want

I think the speed control should have another layout in landscape mode, so most users don't have to scroll.
There is an image how it could look

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

At the moment you have to scroll through the menu and see not all options at the same time.
I have to admit that my phone screen may be a bit thin and long but i think it is the same issue on many other devices that the menu is too big in vertical and has enough space left on the sites.

Additional context

At the time:

My suggestion:

How will you/everyone benefit from this feature?

Its just a detail, but i think it would improve UX because you see everything after opening it up and don't have to scroll anymore - it would just feel cleaner.

[MD77MD]

good catch😎

[Hugov8]

Hello, we are a group of students, can we work on this issue ?

[defini-tiv]

If you ask me: of course!
I think many people would benefit and i would be very thankful by my self.

[litetex]

After reviewing #7351 I want to note a few things that should be considered:

• Having 2 layouts causes 2x maintenance effort.
  There have been similar problems in the past when something was added to player: Someone forgot to add a button on the large-player and the whole app crashed on TV and tablets.
• The dialog is adjusted to the device. Some devices will always have this problem particularly small/old devices.
• With different languages the required text-area sometime gets larger. Which then also causes problems.

So I would keep it stupid simple:
Keep one layout and remove some useless spaces/margins from that layout.

I did that with a few simple changes and I think the results are actually pretty good:

Update: Removed the (obvious) header (see #7363 (comment))

[litetex]

Created a PR for my implementation: #7363