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.

Sign up for GitHub

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

Jump to bottom

WebUI: Add color scheme switcher (v5_0_x) #21748

Merged
merged 2 commits into from
Nov 9, 2024
Merged

WebUI: Add color scheme switcher (v5_0_x) #21748

merged 2 commits into from
Nov 9, 2024

Conversation

sledgehammer999
Copy link
Member

Original PR: #21613

The css is different in this branch.
I took the liberty of changing the * selector to the root selector. I hope this is fine.

@sledgehammer999 sledgehammer999 added WebUI WebUI-related issues/changes Backport labels Nov 2, 2024
@sledgehammer999 sledgehammer999 added this to the 5.0.2 milestone Nov 2, 2024
@HanabishiRecca
Copy link
Contributor

We probably also would need to pull the fixup #21750.

Chocobo1
Chocobo1 approved these changes Nov 3, 2024
View reviewed changes
Copy link
Member

@Chocobo1 Chocobo1 left a comment

Choose a reason for hiding this comment

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

LGTM

@sledgehammer999 sledgehammer999 merged commit 1e27e65 into qbittorrent:v5_0_x Nov 9, 2024
14 checks passed
@sledgehammer999 sledgehammer999 deleted the backport_webui_color_switcher branch November 9, 2024 10:15
@HanabishiRecca HanabishiRecca mentioned this pull request Nov 9, 2024
Merged
Chocobo1 pushed a commit that referenced this pull request Nov 11, 2024
@HanabishiRecca
WebUI: fix color scheme for iframes
9ab3c57 
A backport of #21750 as a follow up to #21748.

Original PR #21750.
PR #21810.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Chocobo1 Chocobo1 Chocobo1 approved these changes

Assignees
No one assigned
Labels
Backport WebUI WebUI-related issues/changes
Projects
None yet
Milestone
5.0.2
Development

Successfully merging this pull request may close these issues.
3 participants
@sledgehammer999 @HanabishiRecca @Chocobo1