Bug Tracker

Opened 6 years ago

Closed 5 years ago

#8401 closed enhancement (fixed)

minor enhancement to opacity cssHooks

Reported by: lrbabe Owned by: jitter
Priority: low Milestone: 1.6
Component: css Version: 1.5.1
Keywords: Cc:
Blocked by: Blocking:

Description

Why are we doing 

var filter = style.filter || "";
style.filter = style.filter + ' ' + opacity;

When we could do 

var filter = style.filter || "";
// reuse the 'filter' var
style.filter = filter + ' ' + opacity;

Improve jQuery, one DOM access at a time.

The actual code is slightly longer, but the modification is equally tiny, see following push request.

Change History (3)

comment:1 Changed 6 years ago by lrbabe

Here is the pull request: ​https://github.com/jquery/jquery/pull/254/files

comment:2 Changed 6 years ago by jitter

  • Component changed from unfiled to css
  • Owner set to jitter
  • Priority changed from undecided to low
  • Status changed from new to assigned

comment:3 Changed 5 years ago by john

  • Milestone changed from 1.next to 1.6
  • Resolution set to fixed
  • Status changed from assigned to closed

Landed.

Note: See TracTickets for help on using tickets.