Replace the id with a class. It's less risky.
https://skitch.com/jesse.beach/fkkf1/netbeans

Move the new class into a template variable and print out the variable.

CommentFileSizeAuthor
#1 convert-aggregator-wrapper-html5-1221712.001.patch466 byteskarschsp
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

karschsp’s picture

karschsp CreditAttribution: karschsp commented
Status: Active » Needs review
FileSize
convert-aggregator-wrapper-html5-1221712.001.patch466 bytes

It looks like this was converted to a class somewhere else, and the aggregator-wrapper is already in $classes, so I just changed one line:

<div class="aggregator">

to

<div class="<?php print $classes; ?>">

Status: Needs review » Needs work

The last submitted patch, convert-aggregator-wrapper-html5-1221712.001.patch, failed testing.

karschsp’s picture

karschsp CreditAttribution: karschsp commented
Status: Needs work » Needs review

Oh testbot, you're so funny. You tell me one thing but you mean another.

Try again?

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Status: Needs review » Reviewed & tested by the community

Make sense

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Makes sense to me too. Committed/pushed to 8.x.

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

updated the description to include moving the class to a template variable.