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.
Unnecessary white space in session-active class in language.negotiation.inc
This can make other modules looking for that class harder.
Comment | File | Size | Author |
---|---|---|---|
#1 | drupal-2147321-1-unnecessary_space.patch | 937 bytes | manfer |
Comments
Comment #1
manfer CreditAttribution: manfer commentedComment #2
manfer CreditAttribution: manfer commentedSame in drupal 7 in locale.inc
Comment #4
manfer CreditAttribution: manfer commented1: drupal-2147321-1-unnecessary_space.patch queued for re-testing.
Comment #6
manfer CreditAttribution: manfer commentedComment #7
manfer CreditAttribution: manfer commented1: drupal-2147321-1-unnecessary_space.patch queued for re-testing.
Comment #9
manfer CreditAttribution: manfer commentedComment #12
manfer CreditAttribution: manfer commented1: drupal-2147321-1-unnecessary_space.patch queued for re-testing.
Comment #13
manfer CreditAttribution: manfer commentedComment #14
droplet CreditAttribution: droplet commentedPatch do not apply
Comment #15
droplet CreditAttribution: droplet commentedNo more in D8
Comment #16
droplet CreditAttribution: droplet commented2: drupal7-2147321-unnecessary_space.patch queued for re-testing.
Comment #17
jcovington CreditAttribution: jcovington commentedIs that whitespace actually, in some cases, necessary? So that the session-active class can be added to an element that in some cases may already have a class or classes associated with it?
Ex. class="some-class some-other-class session-active"
Comment #18
droplet CreditAttribution: droplet commentedNo, Drupal will add space into it :)
Comment #19
smussbach CreditAttribution: smussbach commentedI can confirm that the patch works at current dev.
Comment #20
David_Rothstein CreditAttribution: David_Rothstein commentedCommitted to 7.x - thanks!