Skip to content

[player] Fix non-fullscreen layout regression#12711

Closed
Ljn0626 wants to merge 4 commits intoTeamNewPipe:devfrom
Zer0tier:fix/player-topbar-alignment
Closed

[player] Fix non-fullscreen layout regression#12711
Ljn0626 wants to merge 4 commits intoTeamNewPipe:devfrom
Zer0tier:fix/player-topbar-alignment

Conversation

@Ljn0626
Copy link
Copy Markdown
Contributor

@Ljn0626 Ljn0626 commented Oct 16, 2025

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Removed binding.metadataView.setVisibility(isFullscreen ? View.VISIBLE : View.GONE);
  • Updated the layout: increased audioTrackTextView’s layout_weight to 2

Before/After Screenshots/Screen Record

  • Before: When not in fullscreen, all other items are no longer correctly aligned to the right side
d0fe398db90d754ac767d660b36e93e1 d402a1fc047c57f57e53ec33e531961f
  • After: It displays correctly on both small and medium phones.
f430308fb6caca7933f78743bf916420 7888971cefa2933987f1eea5b2f94b3d 1490d4a0e37940392e841096ceb76e0e f671e3ac3c7eb4c3380631105b8a8685 e26ffb8310ee0e41b94fafaf0244aaaf

Fixes the following issue(s)

  • Follow-up to Issue#12499

Relies on the following changes

  • android:layout_weight="2" ->android:layout_weight="2"
  • delete binding.metadataView.setVisibility(isFullscreen ? View.VISIBLE : View.GONE);

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. You can find more info and a video demonstration on this wiki page.

Due diligence

… : View.GONE);and changed layout_weight of audioTrackTextView to 2
@github-actions github-actions Bot added the size/small PRs with less than 50 changed lines label Oct 16, 2025
@Stypox
Copy link
Copy Markdown
Member

Stypox commented Oct 16, 2025

Thanks! Now the language chooser does not seems to expand to take up as much space as it needs though

@litetex litetex self-requested a review October 16, 2025 18:23
@litetex litetex self-assigned this Oct 16, 2025
@litetex
Copy link
Copy Markdown
Member

litetex commented Oct 16, 2025

There are still layouting problems as can be seen above in the screenshots.

After doing some experiments I think it's impossible to fix this properly without introducing at least one layout that wraps the metadataView and audioTrackTextView.
grafik

Otherwise the layouts constantly "fight" each other or leave some empty, unused spaces:
grafik
grafik

I will try to have a look into this again tomorrow...

PS: We should also fix the issue/PR image resizer because my mouse wheel is now on fire.

@Ljn0626
Copy link
Copy Markdown
Contributor Author

Ljn0626 commented Oct 17, 2025

I’m not sure if this makes sense, but I added an empty <View> as a spacer.
It only becomes visible (with weight) when both metadataView and audioTrackTextView are not present.
This is to ensure that audioTrackTextView can expand to take up as much space as it needs when visible, while keeping the right-side buttons properly aligned.
6a767910428ce77c92d16cf006390401
bb61955e35ed29e8ad2d6269d23dd90b
e999a6080e762f172d686c65fa8d0f4e
4fe28c4dc0f6d3bcd76393b8b7759d79

@litetex
Copy link
Copy Markdown
Member

litetex commented Oct 17, 2025

I think we can close this in favor of my PR

@litetex litetex removed their assignment Oct 17, 2025
@litetex litetex removed their request for review October 17, 2025 19:35
@Stypox
Copy link
Copy Markdown
Member

Stypox commented Oct 21, 2025

Closing in favour of #12714. Thanks anyway! :-)

@Stypox Stypox closed this Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/small PRs with less than 50 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants