Follow up to #2389735: Core and base theme CSS files in libraries override theme CSS files with the same name

More background from @neclimdul.
#1920902: Add a Drupal Yaml wrapper so we can default to PECL Yaml component if it is available
https://www.drupal.org/node/2135307

According to the YAML spec (http://www.yaml.org/spec/1.2/spec.html) the @ symbol is a reserved character, but we've used it for namespace tokens. A solution is to quote the string.

This issue fixes it in the related theme system info files.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task
Issue priority Normal priority
Unfrozen changes Unfrozen because it only changes theme info files. No apis are changed.
Disruption This should not be disruptive.
CommentFileSizeAuthor
add-quote-theme-yaml.diff2.43 KBdavidhernandez
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davidhernandez’s picture

We should probably adjust the related change record. https://www.drupal.org/node/2473869

davidhernandez’s picture

Issue summary: View changes
star-szr’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

Thanks for this '@davidhernandez', I updated the change record and confirmed the patch gets them all!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Cool, this should be a harmless change.

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 041fee6 on 8.0.x
    Issue #2481183 by davidhernandez: Quote tokens in theme .info.yml files
    

Status: Fixed » Closed (fixed)

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