Bug Tracker

Opened 6 years ago

Closed 5 years ago

#9842 closed enhancement (wontfix)

.css(property, null) should work like .css(property, '')

Reported by: borislav.asdf@… 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 6 years ago by Rick Waldron

Component: unfiledcss
Milestone: 1.next1.7
Owner: set to Rick Waldron
Priority: undecidedlow
Status: newassigned

comment:2 Changed 6 years ago by Rick Waldron

Milestone: 1.71.next

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

comment:3 Changed 5 years ago by dmethvin

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.

Note: See TracTickets for help on using tickets.