Bug Tracker


Ticket #7974 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

Cleanup effects.js to improve performances and reduce filesize

Reported by: lrbabe Owned by:
Priority: low Milestone: 1.6
Component: effects Version: 1.4.4
Keywords: Cc:
Blocking: Blocked by:


I've opened a  pull request of small changes to effect.js that:

  • reduce function calls, jQuery.now() calls in particular
  • reduce number of out-of-scope variables lookups
  • cache more variables to reduce file-size once minified

It yields some interesting performance improvement, as demonstrated in this example:  http://jsfiddle.net/louisremi/V3Xqt/5/ (remove patched effects.js in "manage ressources" menu to see the difference)

  • function calls: 1177 -> 1022 = 13% better
  • average execution time: 26.1 -> 23.4 = 10% better

/!\ example might be unrealistic /!\

As a side effect, it keeps animations synchronized and thus fixes #7917 and addresses a similar problem to #6281

Change History

comment:1 Changed 3 years ago by jitter

  • Priority changed from undecided to low
  • Type changed from bug to enhancement
  • Component changed from unfiled to effects
  • Milestone changed from 1.next to 1.6

comment:2 Changed 3 years ago by snover

  • Status changed from new to open

comment:3 Changed 3 years ago by lrbabe

Additional benefits with last commits:

  • reduce function calls even more
  • simplifies easing resolution code
  • add some comments and improve code readability here and there

comment:4 Changed 3 years ago by john

  • Status changed from open to closed
  • Resolution set to fixed

This particular pull request was closed - a new open was opened up (and closed) just recently.

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


Add a comment

Modify Ticket

as closed

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

Note: See TracTickets for help on using tickets.