- Issue created by @penyaskito
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
This looks quite opinionated. We might want to change these based on something like the menu name including cta in its machine name?
For now we will apply this to every menu, but is quite plausible we want to revisit this. - Merge request !31Issue #3487504 by penyaskito: Render menus as a list of CTAs → (Merged) created by penyaskito
- 🇪🇸Spain plopesc Valladolid
Styles are applied properly and conditional logic for Gin theme is working as well.
Using Claro:
Using Gin:
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
We can reuse the existing PR for the default/gin theming switching.
But the styling is gonna change, and we'd like it to look like /admin/config
For the defaults we probably want to use the claro styling for that same page. -
penyaskito →
committed 3627f2c7 on 2.x
Issue #3487504: Styling menus for claro (default) and gin
-
penyaskito →
committed 3627f2c7 on 2.x
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Merged! Claro looks great!
Let's create a new one for gin. @penyaskuto, this is ready for review including Gin accent colors and dark mode options.
-
penyaskito →
committed 45e60b24 on 2.x authored by
javi-er →
Issue #3487504 by javi-er: Styling menus for claro (default) and gin
-
penyaskito →
committed 45e60b24 on 2.x authored by
javi-er →
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
This looks ***awesome***. Didn't expect dark mode to be honest.
Automatically closed - issue fixed for 2 weeks with no activity.