Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Twig Templates to Copy
- core/modules/system/templates/admin-block-content.html.twig
- core/modules/system/templates/admin-block.html.twig
- core/modules/system/templates/admin-page.html.twig
Comment | File | Size | Author |
---|---|---|---|
#3 | interdiff.txt | 3.1 KB | seantwalsh |
#3 | 2407713-3.patch | 1.78 KB | seantwalsh |
#2 | issue-2407713-2.patch | 4.8 KB | Sivaji_Ganesh_Jojodae |
Comments
Comment #1
Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commentedComment #2
Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commentedComment #3
seantwalshRemoved the "set classes" in core/modules/system/templates/admin-block-content.html.twig
Comment #4
davidhernandezPostponing this until we decide how we are moving forward. Keep an eye on #2348543: [meta] Consensus Banana Phase 2, transition templates to the starterkit theme for updates. Thanks.
Comment #5
mortendk CreditAttribution: mortendk commentedstill postponed ? ;)
Comment #6
mortendk CreditAttribution: mortendk commentedComment #7
mortendk CreditAttribution: mortendk commentedNone of these templates we copied over to classy on the big merge, so we need to take a desission if we want to move em over.
Mistakenly made an duplicated issue on this with this summary that i think summons up the problem, were getting if we dont clean up & move ove
Not moving templates over based on the assumption that nobody wanna do an admin theme & therefor it should not be in classy is gonna burn us down the road.
- classy is where all classes that drupal uses for visual elements as decided by the banana concensus
- it makes a false assumption of "what a themer might wanna do" (yes an admin theme needs love to)
- update is one of the templates / css that any admin theme is gonna use a lot of time
- makes it harder for seven & other admin themes to know whats the exact markup is & why isnt that in classy
- Drupals default admin theme is seven & is building on classy, having a strange seperation all the way into stark, cause its "admin" makes no logical sense
- Admin themes need's love & classy to
Comment #8
lauriiiComment #9
joelpittetAutomated triage: bumping to 9.0.x because our API base themes (stable and classy) are locked in with the release of 8.0.x.
Comment #10
catchComment #23
quietone CreditAttribution: quietone at PreviousNext commentedAsked in #frontend and andy-blum replied that this should move to the contrib project.