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.
(First, verify that the preprocess changes have been made. #2322163: [meta] Consensus Banana Phase 1, move CSS classes from preprocess to twig templates.)
- Copy the Twig templates from the core module's templates directory to Classy's templates directory. Include all templates, even ones without classes.
- Remove all classes from the core module's template. Remove all classes added with addClass and ones that are hard-coded in the template.
- If there are classes that are required for basic functionality, discuss whether they should be kept.
- If there is CSS from the module, or anywhere else, referring to the class, discuss removing it or moving it to Bartik&Seven. Do not move the CSS to Classy.
Twig Templates to Copy
core/modules/rdf/templates/rdf-metadata.html.twig
Comment | File | Size | Author |
---|---|---|---|
#10 | interdiff.txt | 1.15 KB | Xen |
#10 | copy_rdf_templates_to-2349727-10.patch | 707 bytes | Xen |
#4 | copy_rdf_templates_to-2349727-4.patch | 1.35 KB | cilefen |
#4 | interdiff-2-4.txt | 1.58 KB | cilefen |
#2 | copy_rdf_templates_to-2349727-2.patch | 1.36 KB | cilefen |
Comments
Comment #1
cilefen CreditAttribution: cilefen commentedComment #2
cilefen CreditAttribution: cilefen commentedComment #3
davidhernandezThe template should go directly into the templates folder, not a subfolder.
Comment #4
cilefen CreditAttribution: cilefen commentedComment #5
mortendk CreditAttribution: mortendk commentedComment #6
Xen CreditAttribution: Xen commentedComment #7
Xen CreditAttribution: Xen commentedWorks. Is used when using Bartik.
Comment #8
alexpottAre we sure that removing the hidden class is desirable here?
Comment #9
Xen CreditAttribution: Xen commentedNo, you're right. I'll reroll.
Comment #10
Xen CreditAttribution: Xen commentedThere...
Comment #11
Xen CreditAttribution: Xen commentedComment #12
mortendk CreditAttribution: mortendk commentedComment #13
davidhernandezI'm universally setting all the phase 2 issues back to needs review, because we're missing some things. Please double-check if any removed classes are being used in javascript. It is best to test the affected template using Stark to make sure nothing is broken.
Comment #14
mortendk CreditAttribution: mortendk commentedno js affected
Comment #15
alexpottThe class
rdf-meta
is not used anywhere.Committed b913cea and pushed to 8.0.x. Thanks!
This normal task was committed due the fact the banana consensus received per approval wrt to #2350615: [policy, no patch] What changes can be accepted during the Drupal 8 beta phase?