Closed (duplicate)
Project:
Drupal core
Version:
x.y.z
Component:
javascript
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
20 May 2006 at 16:43 UTC
Updated:
21 May 2006 at 09:17 UTC
Jump to comment: Most recent file
I noticed that the removeClass() function in drupal.js does also remove spaces between two class names. The attached patch fixes that by replacing the class with leading and or trailing spaces with $1 instead of an empty string (you could also use $2). If $1 is empty, the class that should be removed is at the beginning of the className string, thus it doesn't matter if the trailing space is also removed. Same goes for the end. Only if the class is somehwere in the middle, a space is left to separate the two neighboring classes.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | drupal.js_8.patch | 567 bytes | kkaefer |
| drupal.js_7.patch | 566 bytes | kkaefer |
Comments
Comment #1
kkaefer commentedSorry, old patch. Here is the new one.
Comment #2
dries commentedTim, your patch looks like a duplicate of #58956. Can you compare both patches?