Bug Tracker

Modify

Ticket #8401 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

minor enhancement to opacity cssHooks

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

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

comment:1 Changed 3 years ago by lrbabe

comment:2 Changed 3 years ago by jitter

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

comment:3 Changed 3 years ago by john

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

Landed.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.