Skip to content

VideoDetailFragment: composfiy views and thumbs chunck#13273

Open
dustdfg wants to merge 1 commit intoTeamNewPipe:refactorfrom
dustdfg:likes_compose
Open

VideoDetailFragment: composfiy views and thumbs chunck#13273
dustdfg wants to merge 1 commit intoTeamNewPipe:refactorfrom
dustdfg:likes_compose

Conversation

@dustdfg
Copy link
Contributor

@dustdfg dustdfg commented Feb 21, 2026

What is it?

  • Bugfix (user facing)
  • Feature (user facing) ⚠️ Your PR must target the refactor branch
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • VideoDetailFragment: composfiy views and thumbs chunck

Before/After Screenshots/Screen Record

20260221_16h22m19s_grim
  • Before:
20260221_16h23m59s_grim
  • After:
20260221_16h24m36s_grim

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

@github-actions github-actions bot added the size/large PRs with less than 750 changed lines label Feb 21, 2026
@dustdfg
Copy link
Contributor Author

dustdfg commented Feb 21, 2026

Well, this looks more like original view based version then previous PR and uses styles rather than plain fontSizes. I couldn't normally convert channel part so basically it is copy of previous PR with mentioned changes (github didn't let me reopen that PR after forcepush)...

alignment = LineHeightStyle.Alignment.Proportional,
trim = LineHeightStyle.Trim.LastLineBottom
)
)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't ask me about it. Idk. It helped the text box to be smaller and look more like original view based version. Why, how, idk

@ShareASmile ShareASmile added the rewrite Issues and PRs related to rewrite label Feb 24, 2026
@github-project-automation github-project-automation bot moved this to In Progress in Rewrite Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rewrite Issues and PRs related to rewrite size/large PRs with less than 750 changed lines

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants