infrastructure/conference-website
1
1
Fork 1
Code Issues 14 Pull Requests Releases Activity

UI: Implement Web Assets' theme system, and add 'dark' theme #103972

Merged
Márton Lente merged 23 commits from ui/theme-system into main 2024-09-30 15:03:39 +02:00
Conversation 15 Commits 23 Files Changed 13 -4
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 69b230eaa7 - Show all commits

4
conference_main/static/conference_main/styles/main.sass
View File

@ -269,10 +269,6 @@ button.going-star
.form-check-input .form-check-input
transform: scale(1.5) translateX(calc(var(--spacer) * -1)) transform: scale(1.5) translateX(calc(var(--spacer) * -1))
// TODO: remove rule after Web Assets v2 component nav-pills has been implemented
.list-group-item
background-color: var(--box-bg-color)
// TODO: check and cleanup style definition in Web Assets // TODO: check and cleanup style definition in Web Assets
.table-no-box .table-no-box
martonlente marked this conversation as resolved Outdated
Pablo Vazquez commented 2024-09-26 12:26:13 +02:00
Outdated
Review
Copy Link

Hasn't this be implemented already? 08054cfd5f

Hasn't this be implemented already? https://projects.blender.org/infrastructure/web-assets/commit/08054cfd5f99c9097da9ec19ea4e5fe5156bd6e3
Márton Lente commented 2024-09-26 13:08:15 +02:00
Outdated
Review
Copy Link

Thanks for the reference. It was added to web-assets, but not yet implemented to the Conference website at the time of the commit. Now it's present on main, and I cleaned up the style in the PR that became redundant in the meanwhile. 👍

Thanks for the reference. It was added to web-assets, but not yet implemented to the Conference website at the time of the commit. Now it's present on `main`, and I cleaned up the style in the PR that became redundant in the meanwhile. 👍
tr tr