Skip to main content

Bug Tracker

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 rwaldron comment:1

component: unfiledcss
milestone: 1.next1.7
owner: → rwaldron
priority: undecidedlow
status: newassigned

Changed September 07, 2011 09:13PM UTC by rwaldron comment:2

milestone: 1.71.next

No discussion was had in a reasonable amount of time for this ticket.

Changed July 14, 2012 02:11AM UTC by dmethvin comment:3

resolution: → wontfix
status: assignedclosed

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.