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

Switch away from SASS (also new themes!) #386

Merged
merged 11 commits into from
Dec 19, 2024
Merged

Switch away from SASS (also new themes!) #386

merged 11 commits into from
Dec 19, 2024

Conversation

Meow
Copy link
Member

@Meow Meow commented Dec 17, 2024

This commit removes SASS/SCSS, and replaces it with PostCSS with Relative Color Syntax, Simple Variables and Mixin plugins. Significantly reduces frontend asset build times (I measured a decrease from 1.5-3 seconds to 600 milliseconds on my machine).

Also introduces many new themes. Old themes are translated as such:

old new
default light-blue
dark dark-blue
red dark-red

Additionally, themes are now separated into two categories, dark and light. Each category contains the following color variations: red, orange, blue, yellow, green, purple, cyan, pink, silver/charcoal

dark-purple is as seen on furbooru.org
light-green is as seen on manebooru.art

lib/philomena/users/user.ex Outdated Show resolved Hide resolved
lib/philomena_web/templates/setting/edit.html.slime Outdated Show resolved Hide resolved
lib/philomena_web/views/layout_view.ex Show resolved Hide resolved
lib/philomena_web/views/setting_view.ex Outdated Show resolved Hide resolved
lib/philomena_web/views/setting_view.ex Outdated Show resolved Hide resolved
lib/philomena_web/views/setting_view.ex Outdated Show resolved Hide resolved
assets/css/elements/base.css Show resolved Hide resolved
assets/css/elements/forms.css Show resolved Hide resolved
assets/css/views/images.css Show resolved Hide resolved
@Meow Meow merged commit 66de088 into master Dec 19, 2024
8 checks passed
@liamwhite liamwhite deleted the postcss branch December 19, 2024 18:29
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