Side navigation
#9842 closed enhancement (wontfix)
Opened July 16, 2011 08:34AM UTC
Closed July 14, 2012 02:11AM UTC
.css(property, null) should work like .css(property, '')
| Reported by: | borislav.asdf@gmail.com | Owned by: | rwaldron |
|---|---|---|---|
| Priority: | low | Milestone: | 1.next |
| Component: | css | Version: | 1.6.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
IMO it is not great coding practice to use empty string for something which the language has built in sort of "empty value" - null. The .css() function can set string and integer values so using empty string to remove the value in both cases is a little dirty. Using null-s is cleaner IMO.
Attachments (0)
Change History (3)
Changed July 25, 2011 04:20PM UTC by comment:1
| component: | unfiled → css |
|---|---|
| milestone: | 1.next → 1.7 |
| owner: | → rwaldron |
| priority: | undecided → low |
| status: | new → assigned |
Changed September 07, 2011 09:13PM UTC by comment:2
| milestone: | 1.7 → 1.next |
|---|
No discussion was had in a reasonable amount of time for this ticket.
Changed July 14, 2012 02:11AM UTC by comment:3
| resolution: | → wontfix |
|---|---|
| status: | assigned → closed |
I don't see a need for this, it just adds more special cases when the caller can say what they mean. In most cases you're better off using classes anyway.