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.
A stylesheet from the Classy theme is attached in templates/content/node.html.twig on line 84
{{ attach_library('classy/node') }}
I suppose it’s a copy-and-past leftover.
Comment | File | Size | Author |
---|---|---|---|
#13 | 3157308-13.patch | 1.31 KB | hansa11 |
#8 | 3157308-8.patch | 502 bytes | steinmb |
#5 | interdiff_2-5.txt | 763 bytes | kishor_kolekar |
#5 | 3157308-5.patch | 540 bytes | kishor_kolekar |
Comments
Comment #2
riccardoR CreditAttribution: riccardoR commentedProvided patch changes the line as follows:
{{ attach_library('olivero/node') }}
Comment #3
steinmb CreditAttribution: steinmb as a volunteer commentedDoes not seem right. Patch apply cleanly
And addresses the issue. RTBC
Comment #4
mherchelThis still needs to be addressed.
Comment #5
kishor_kolekar CreditAttribution: kishor_kolekar as a volunteer and at QED42 for Drupal India Association commentedplease review the patch.
Comment #6
steinmb CreditAttribution: steinmb as a volunteer commentedThe patch apply correctly though I have a question:
When I look in
core/themes/olivero/olivero.libraries.yml
I do not seenode:
library defined. Am I confused (as usual)?Comment #7
mherchelYou are correct. There is no
olivero/node
library.Comment #8
steinmb CreditAttribution: steinmb as a volunteer commentedQuick one liner.
Comment #9
mherchelThis is correct, however the classy/node library references one CSS file which contains one CSS rule that we need to duplicate in Olivero:
We need to add this to our node.pcss.css file, and add a comment stating that there is no variable for this color.
Comment #10
hansa11 CreditAttribution: hansa11 as a volunteer and at Axelerant commentedComment #11
hansa11 CreditAttribution: hansa11 as a volunteer and at Axelerant commentedPatch details:
Removed the classy library from Olivero's node.html.twig
Added the required CSS in node.pcss.css
Please review.
Comment #12
mherchelThis is looking soooo close!
Need to insert a space before the beginning and after the end of the comment. Also need to add a period at the end of the sentence.
We need to apply coding standards to the comment (https://www.drupal.org/docs/develop/standards/css/css-formatting-guideli...)
Comment #13
hansa11 CreditAttribution: hansa11 as a volunteer and at Axelerant commented@mherchel: Thank you for the review.
Updated the patch as per your suggestion, please review.
Thanks!
Comment #14
hansa11 CreditAttribution: hansa11 as a volunteer and at Axelerant commentedComment #15
mherchelThis looks good to me. Thanks!
Comment #16
alexpottCommitted and pushed bf29675611 to 9.2.x and 7d8ea2dccb to 9.1.x. Thanks!