[wp-trac] [WordPress Trac] #25112: In large menus javascript freezes the browser and you cannot work with the menu.

[WordPress Trac]

#25112: In large menus javascript freezes the browser and you cannot work with the
menu.
-------------------------------------------------+-------------------------
 Reporter:  tsdexter                             |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  3.7
Component:  Menus                                |     Version:  3.6
 Severity:  blocker                              |  Resolution:
 Keywords:  ux-feedback ui-feedback 2nd-opinion  |
  has-patch needs-testing                        |
-------------------------------------------------+-------------------------

Comment (by atimmer):

 Changing menuItemDepth in
 [http://core.trac.wordpress.org/attachment/ticket/25112/25112.1.diff
 25112.1.diff] was too aggressive. It is used to put the classes there in
 the first place.

 [http://core.trac.wordpress.org/attachment/ticket/25112/25112.2.diff
 25112.2.diff] is a new patch without those changes but with new changes.
 show() and hide() are heavy operations because they have to check the
 'display' css property of the items they are going to show/hide. And that
 is heavy when used in a loop with a lot of iterations.

 I replaced those calls with the equivalent, putting display on
 inline/hidden.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25112#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software