Opened 12 years ago
Closed 11 years ago
#9842 closed enhancement (wontfix)
.css(property, null) should work like .css(property, '')
Reported by: | Owned by: | Rick Waldron | |
---|---|---|---|
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.
Change History (3)
comment:1 Changed 12 years ago by
Component: | unfiled → css |
---|---|
Milestone: | 1.next → 1.7 |
Owner: | set to Rick Waldron |
Priority: | undecided → low |
Status: | new → assigned |
comment:2 Changed 11 years ago by
Milestone: | 1.7 → 1.next |
---|
comment:3 Changed 11 years ago by
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.
Note: See
TracTickets for help on using
tickets.
No discussion was had in a reasonable amount of time for this ticket.