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.
Problem/Motivation
Conditional comments are only supported by IE <=9, and any core libraries that make use of them are already removed (see https://www.drupal.org/node/3089508)
Proposed resolution
Remove HtmlTag::preRenderConditionalComments()
Remaining tasks
User interface changes
API changes
The 'browsers'
attribute of library assets is ignored.
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#14 | 3101620-14.patch | 29.43 KB | longwave |
| |||
#12 | 3101620-12.patch | 29.76 KB | longwave |
| |||
#4 | drupal-3101620-4.patch | 21.29 KB | gapple |
Comments
Comment #2
gappleComment #4
gappleLast patch missed use of
'browsers'
inCssCollectionGrouper
Comment #5
gappleComment #6
gappleComment #7
gappleOne consideration is that if a site has missed or ignored prior warning about
'browsers'
being deprecated and not updated to remove any resources that were restricted to IE <= 9, then this change will cause those resources to be output unconditionally for all browsers.To prevent this,
HtmlTag
could render an empty string if'browsers'
is set to anything other than['IE' => TRUE, '!IE' => TRUE]
if it exists.Comment #8
andypostComment #9
gappleThe corresponding deprecation needs to be delayed until 9.1, so removal can't happen till 10.0
Comment #10
gappleComment #11
gappleDeprecation has been committed to 9.1
Comment #12
longwaveRerolled #4 and found a few more bits to clean up.
Comment #13
gappleSetting parent to meta issue
Comment #14
longwaveRerolled.
Comment #15
andypostComment #16
catchPatch looks great - just removals of deprecated code paths.
Comment #18
lauriiiCommitted 2307e12 and pushed to 10.0.x. Thanks!