Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

style: add theme support #550

Closed
wants to merge 8 commits into from
Closed

style: add theme support #550

wants to merge 8 commits into from

Conversation

bumi
Copy link

@bumi bumi commented Jul 1, 2024

this adds themes using different base variables..

Note: I don't know what I am doing

@bumi bumi requested review from rolznz and reneaaron July 1, 2024 08:11
@rolznz
Copy link
Collaborator

rolznz commented Jul 1, 2024

image

dropdown placement / size does not look so good. Maybe @reneaaron can do his magic 🪄

Copy link
Collaborator

@rolznz rolznz left a comment

Choose a reason for hiding this comment

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

tACK

@reneaaron
Copy link

I fixed the settings layout and generally like the themes, however since we introduced some special variables (warnings, success, etc) and the generated themes do not include these variables I was thinking to reduce the amount of themes available and polish the ones we provide by adding proper values for these variables as well.

@im-adithya
Copy link
Member

Also can add a card somewhere in the UI to tell the user how to set themes (and in README as well)

cc @stackingsaunter

@reneaaron reneaaron closed this Jul 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants