Skip to content

Conversation

@Sp00kyTec
Copy link

@Sp00kyTec Sp00kyTec commented Sep 13, 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

  • record videos
  • create clones
  • take over the world

Before/After Screenshots/Screen Record

  • Before:
  • After:

Fixes the following issue(s)

  • Fixes #

Relies on the following changes

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/giant PRs with more than 750 changed lines label Sep 13, 2025
@Isira-Seneviratne Isira-Seneviratne added the template ignored The user didn't follow the template/instructions (or removed them) label Sep 14, 2025
@Stypox
Copy link
Member

Stypox commented Sep 15, 2025

Hello, thank you for your PR! NewPipe is currently undergoing a refactor on the refactor branch, and we are rewriting the UI there, and not accepting any more non-strictly-necessary PRs on the dev branch. We are switching from XML to Jetpack Compose there, so it would be good if you could maybe rewrite some simple components in Compose on the refactor branch, and while at it also embrace Material 3 (which is the default on that branch)!

@Isira-Seneviratne
Copy link
Member

Since I have raised a PR for the Material 3 migration, I will close this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/giant PRs with more than 750 changed lines template ignored The user didn't follow the template/instructions (or removed them)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants