Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: landscape layout for player page #4981

Merged
merged 5 commits into from
Oct 23, 2023

Conversation

Bnyro
Copy link
Member

@Bnyro Bnyro commented Oct 14, 2023

closes #3198

@Bnyro Bnyro marked this pull request as ready for review October 19, 2023 19:08
@Bnyro Bnyro added help wanted Looking for help testers needed Looking for people to test the changes before merging labels Oct 19, 2023
@Bnyro Bnyro changed the title feat: landscape layout for player page [WIP] feat: landscape layout for player page Oct 20, 2023
@rozari0
Copy link
Contributor

rozari0 commented Oct 21, 2023

When I open Settings Libretube launches in portrait mode first and then rotate to landscape mode.
Otherwise works fine.

Tested on Android 9.

@Bnyro Bnyro changed the title [WIP] feat: landscape layout for player page feat: landscape layout for player page Oct 23, 2023
@Bnyro Bnyro merged commit fc9e3e6 into libre-tube:master Oct 23, 2023
2 of 3 checks passed
@Bnyro Bnyro deleted the landscape-player-page branch October 23, 2023 15:52
@Baardi
Copy link

Baardi commented Nov 8, 2023

Hi.

The tiling might not be working out as you planned it. This is from your debug build from today, from a Samsung Galaxy Tab S5e (Android 11). Ignore the slightly off aspect ratio, it looks normal, I just chose a terrible method of screenshotting

Neverthless thank you for an amazing app, and thank you for working on this.

Screenshot_20231108-195840_VLC.jpg

Screenshot_20231108-195901_VLC.jpg

@Bnyro
Copy link
Member Author

Bnyro commented Nov 8, 2023

Hmm, do you have related streams disabled?

@Bnyro
Copy link
Member Author

Bnyro commented Nov 8, 2023

Oh, I can reproduce the issue now. Seems like this was introduced a few commits ago, it did work pretty well when I merged this PR.

Please open an issue so that we don't forget to fix this before the next release.

@Baardi
Copy link

Baardi commented Nov 8, 2023

Opened #5131

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Looking for help testers needed Looking for people to test the changes before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Landscape layout for player page
3 participants