Bug Tracker

Opened 5 years ago

Closed 4 years ago

#9842 closed enhancement (wontfix)

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

Reported by: borislav.asdf@… 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.

Change History (3)

comment:1 Changed 5 years ago by rwaldron

  • Component changed from unfiled to css
  • Milestone changed from 1.next to 1.7
  • Owner set to rwaldron
  • Priority changed from undecided to low
  • Status changed from new to assigned

comment:2 Changed 4 years ago by rwaldron

  • Milestone changed from 1.7 to 1.next

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

comment:3 Changed 4 years ago by dmethvin

  • Resolution set to wontfix
  • Status changed from assigned to 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.