(First, verify that the preprocess changes have been made. #2322163: [meta] Consensus Banana Phase 1, move CSS classes from preprocess to twig templates.)

  1. Copy the Twig templates from the core module's templates directory to Classy's templates directory. Include all templates, even ones without classes.
  2. 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

CommentFileSizeAuthor
#10 interdiff.txt1.15 KBxen
#10 copy_rdf_templates_to-2349727-10.patch707 bytesxen
#4 copy_rdf_templates_to-2349727-4.patch1.35 KBcilefen
#4 interdiff-2-4.txt1.58 KBcilefen
#2 copy_rdf_templates_to-2349727-2.patch1.36 KBcilefen

Comments

cilefen’s picture

cilefen commented
Assigned: Unassigned » cilefen
cilefen’s picture

cilefen commented
Assigned: cilefen » Unassigned
Status: Active » Needs review
Issue tags: +Novice
StatusFileSize
new copy_rdf_templates_to-2349727-2.patch1.36 KB
davidhernandez’s picture

davidhernandez
Primary language English
Location USA
commented
Status: Needs review » Needs work

The template should go directly into the templates folder, not a subfolder.

cilefen’s picture

cilefen commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-2-4.txt1.58 KB
new copy_rdf_templates_to-2349727-4.patch1.35 KB
mortendk’s picture

mortendk commented
Issue tags: +drupalhagen
xen’s picture

xen commented
Assigned: Unassigned » xen
xen’s picture

xen commented
Assigned: xen » Unassigned
Status: Needs review » Reviewed & tested by the community

Works. Is used when using Bartik.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented 
+++ b/core/modules/rdf/templates/rdf-metadata.html.twig
@@ -18,5 +18,5 @@
-  <span{{ attributes.addClass('rdf-meta', 'hidden') }}></span>
+  <span{{ attributes }}></span>

Are we sure that removing the hidden class is desirable here?

xen’s picture

xen commented
Assigned: Unassigned » xen

No, you're right. I'll reroll.

xen’s picture

xen commented
StatusFileSize
new copy_rdf_templates_to-2349727-10.patch707 bytes
new interdiff.txt1.15 KB

There...

xen’s picture

xen commented
Assigned: xen » Unassigned
mortendk’s picture

mortendk commented
Issue tags: +cssbanana
davidhernandez’s picture

davidhernandez
Primary language English
Location USA
commented
Status: Reviewed & tested by the community » Needs review

I'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.

mortendk’s picture

mortendk commented
Status: Needs review » Reviewed & tested by the community

no js affected

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

The 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?

  • alexpott committed b913cea on 8.0.x 
    Issue #2349727 by cilefen, Xen | davidhernandez: Copy rdf templates to...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.