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.

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

feat(themes): Online Themes CSS Override #3017

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yuna0x0
Copy link

@yuna0x0 yuna0x0 commented Nov 15, 2024

Resolves #1466.

This PR implemented an online theme CSS override feature, which allows user to configure their online theme using CSS override.

Related PR

Vencord/vencord.dev#69

Limitation

  • When there's an @import CSS at-rule in the remote stylesheet, those imported stylesheets' CSS rules won't be visible. It will make things more complex if we want to fetch and parse those stylesheets recursively.

Screenshots

Screenshot 2024-11-16 at 3 23 11 AM Screenshot 2024-11-16 at 3 23 52 AM

@yuna0x0 yuna0x0 force-pushed the online-theme-override branch 5 times, most recently from 241836f to b59f39e Compare November 15, 2024 22:23
@nakoyasha
Copy link
Contributor

I think it'd be better as a modal rather than an inline, inline (at least, from my experience) has always been a lot more messy and hard to use

@yuna0x0
Copy link
Author

yuna0x0 commented Nov 16, 2024

I think it'd be better as a modal rather than an inline, inline (at least, from my experience) has always been a lot more messy and hard to use

I agree with this. Change to modal component.

Screenshot 2024-11-16 at 8 28 26 AM Screenshot 2024-11-16 at 8 28 41 AM

@yuna0x0 yuna0x0 force-pushed the online-theme-override branch 4 times, most recently from eb3d5c1 to 1fa8aa3 Compare November 16, 2024 20:44
@yuna0x0 yuna0x0 force-pushed the online-theme-override branch 12 times, most recently from 8402dd2 to f7d3dd4 Compare November 18, 2024 00:10
@yuna0x0
Copy link
Author

yuna0x0 commented Nov 18, 2024

Added handly icon buttons for quick edit and reset.

vencord-pr-3017-demo.mp4

@yuna0x0 yuna0x0 force-pushed the online-theme-override branch 4 times, most recently from 4c1f7fa to 5d97b5a Compare November 18, 2024 03:30
@yuna0x0 yuna0x0 force-pushed the online-theme-override branch 3 times, most recently from 7f1e4bf to 30c7740 Compare November 24, 2024 04:32
@yuna0x0 yuna0x0 force-pushed the online-theme-override branch 2 times, most recently from 9c6ef58 to cc3af12 Compare November 26, 2024 14:56
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.

[Feature Request] A theme configurator for Vencord/Vesktop
2 participants