Skip to content

feat(YouTube - Swipe controls): Add separate color settings for the brightness and volume bars #5043

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

Merged
merged 6 commits into from
May 28, 2025

Conversation

MarcaDian
Copy link
Contributor

Adds options to separately set the progress bar color for brightness and volume.

@LisoUseInAIKyrios LisoUseInAIKyrios changed the title feat(YouTube - Swipe controls): Separate the swipe color progress bar into brightness and volume feat(YouTube - Swipe controls): Add separate color settings for the brightness and volume bars May 26, 2025
@LisoUseInAIKyrios
Copy link
Contributor

I don't use swipe controls, but when trying this PR I noticed the brightness progress bar initially showed as a negative value and was drawn to the left and outside of the progress bar area.

The issue only showed once, and after I started to swipe the brightness drawing was fixed and wasn't out of place. I could not reproduce the issue again.

I think the initial setting brightness value of -1 was causing the issue, but I'm not sure.

@MarcaDian

This comment was marked as resolved.

@LisoUseInAIKyrios
Copy link
Contributor

LisoUseInAIKyrios commented May 27, 2025

Yes that's what I remember it looked like.

The Android emulator runs AOSP, if needed.

@MarcaDian

This comment was marked as resolved.

@LisoUseInAIKyrios
Copy link
Contributor

I'm not sure how that would fix the UI drawing issue.

It's not a big deal because it's probably been present for a while, and it only shows on the first swipe and never again.

@MarcaDian
Copy link
Contributor Author

MarcaDian commented May 27, 2025

I can reproduce it, if I set the brightness value to -1 in import/export and disable the auto-brightness gesture, then the first swipe down will show -100.
I'll try to fix it now.

@MarcaDian
Copy link
Contributor Author

MarcaDian commented May 27, 2025

This only fixes the displaying the value -100% on the first swipe, now 0% will be displayed but auto-brightness will still be activated (-1), I think this would be more correct than set the value to 0 (0%).

Otherwise, users will be confused why the brightness will go to 0 when they rotate the screen.

@LisoUseInAIKyrios
Copy link
Contributor

Is this ready to go?

@MarcaDian
Copy link
Contributor Author

Yes.

@LisoUseInAIKyrios LisoUseInAIKyrios merged commit 80f50e8 into ReVanced:dev May 28, 2025
1 check passed
@MarcaDian MarcaDian deleted the swipe-colors branch May 28, 2025 10:00
github-actions bot pushed a commit that referenced this pull request May 28, 2025
# [5.25.0-dev.12](v5.25.0-dev.11...v5.25.0-dev.12) (2025-05-28)

### Features

* **YouTube - Swipe controls:** Add separate color settings for the brightness and volume bars ([#5043](#5043)) ([80f50e8](80f50e8))
github-actions bot pushed a commit that referenced this pull request May 29, 2025
# [5.25.0](v5.24.0...v5.25.0) (2025-05-29)

### Bug Fixes

* **Disable Pairip license check:** Change patch to default off ([74b6a94](74b6a94))
* **Hide ADB status:** Resolve app crash on startup ([#5029](#5029)) ([1abebd5](1abebd5))
* **Messenger:** Remove outdated `Disable switching emoji to sticker` patch ([#5044](#5044)) ([7b182ca](7b182ca))
* **Spotify Lite:** Remove obsolete `Enable on demand` patch ([#5046](#5046)) ([4886d47](4886d47))
* **YouTube - GmsCore support:** Restore patch functionality from prior merge ([7686bbe](7686bbe))
* **YouTube - Hide ads:** Hide new type of general ad ([#5004](#5004)) ([37e59d2](37e59d2))
* **YouTube - Open Shorts in regular player:** Do not exit app when pressing back button in regular player ([#5020](#5020)) ([3384f8d](3384f8d))
* **YouTube:** Better handle incorrect duplicate translations ([20abac5](20abac5))
* **Yuka - Unlock premium:** Remove broken patch that is no longer supported ([#5018](#5018)) ([fac6e59](fac6e59))

### Features

* Add `Disable pairip license check` patch ([#4927](#4927)) ([42d2c27](42d2c27))
* **Messenger:** Add `Remove Meta AI` patch ([#4945](#4945)) ([012dff7](012dff7))
* **Prime Video:** Add `Rename shared permissions` patch ([#5049](#5049)) ([80f1fc6](80f1fc6))
* **Spotify:** Add `Fix Facebook login` patch ([#5023](#5023)) ([34932dc](34932dc))
* **Threads:** Hide Ads ([#5064](#5064)) ([3c4cecb](3c4cecb))
* **YouTube - Enable debugging:** Add settings menu to share debug logs ([#5021](#5021)) ([1ec4a88](1ec4a88))
* **YouTube - Settings:** Add a color picker ([#4981](#4981)) ([1e0e398](1e0e398))
* **YouTube - Swipe controls:** Add separate color settings for the brightness and volume bars ([#5043](#5043)) ([80f50e8](80f50e8))
* **YouTube:** Add `Disable haptic feedback` patch ([#5033](#5033)) ([bbe7974](bbe7974))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants