[#CONF-13373] Tree View hangs Internet Explorer when viewing lots of pages at once

Confluence

Tree View hangs Internet Explorer when viewing lots of pages at once

Created: 13/Oct/08 11:54 PM Updated: 01/Dec/08 07:15 PM

Component/s:

Browser Support, Page Ordering, Web Interface - JavaScript/AJAX

Affects Version/s:

2.9.1

Fix Version/s:

2.10

Time Tracking:

Not Specified

Environment:

IE 7.0.5730.13 / Windows XP

Issue Links:

Duplicate

This issue is duplicated by:

~~CONF-13881~~ CONF: unable to move page's location in Safari. Works in Firefox 3.

Reference

This issue relates to:

CONF-11860 Pagetree component causing errors in large spaces

~~CONF-10535~~ Alphabetical page list (under Browse Space) is too slow on large spaces

This issue is related to:

CONF-13779 Tree View should limit the number of pages displayed

Participants:	Agnes Ro [Atlassian], Andrew Prentice, David Yu [Atlassian], Mark Hrynczak [Atlassian], Matthew Jensen [Atlassian], Oleg Lavrovsky, Simon Hill and Vincent Kopa
Since last comment:	5 weeks, 3 days ago
Internal Complexity:	5
Internal Value:	8
Resolution Date:	01/Dec/08 07:15 PM
Labels:	qa-manual
Reviewers:	Andrew Prentice, Mark Hrynczak [Atlassian] and Paul Curren [Atlassian]

Description

« Hide

This issue has different symptoms from CONF-10535 in that:

Alphabetical View is not affected
FireFox is not affected
Problem can be evident with as little as 300 ~ 800 pages.
Occurs when you have "too many" pages visible at one time, either because you have a parent page with lots of children or becuase you have opened lots of parent nodes with a small amount of children.

When you browse a space, and select Tree View, this will cause IE7 to hang. The hang appears to be on the client side only as the contents of the tree looks to be received from the server based on observations with Fiddler. Performing the same action with FireFox works fine. IE consumes the CPU and the memory usage continually grows.

To reproduce the problem, browse our CAC space (currently on 2.9.2-rc1) here: http://confluence.atlassian.com/display/KB/Home
Expand Home. The Knowledge Base has about 870 articles rooted under Home.

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Simon Hill added a comment - 21/Oct/08 07:13 PM

Because this upgrade is required for several other systems/projects we currently have underway, is it possible to get a time frame on when this is likely to be resolved?
Thanks, Simon.

[ Permlink | « Hide ]

Matthew Jensen [Atlassian] added a comment - 21/Oct/08 08:36 PM

Simon, we'll take a look at this and aim to have a fix for the next release.

[ Permlink | « Hide ]

Matthew Jensen [Atlassian] added a comment - 28/Oct/08 11:47 PM

I took a look at this and was able to reproduce fairly easily.

Create a space with 1000 flat pages.
Open the browse space page: /pages/listpages-dirview.action?key=TST
Firefox will pause briefly then be ok, IE will simply stop responding.

Tested on IE6.

[ Permlink | « Hide ]

Matthew Jensen [Atlassian] added a comment - 05/Nov/08 11:01 PM

We've tried a few other JS trees (extjs for example) and they don't seam to crash the browser, even with a largish number of items. We'll take a look to see how they are manipulating the DOM and see what we can do about it.

[ Permlink | « Hide ]

Vincent Kopa added a comment - 07/Nov/08 06:41 AM

Hi,

I also encounter the 100% CPU problem with IE (6 and 7) when the tasklist macro is displayed on a page.
I guess the javascript used poses a generic problem with IE for the pagetree and the tasklist macros.

Regards,
Vincent

[ Permlink | « Hide ]

Oleg Lavrovsky added a comment - 10/Nov/08 10:13 AM

I can confirm that the tree control freezes IE 6 (XP) at about 300 pages, with the browser noticeably sluggish after about 100 child pages expanded underneath a single parent.

The control in the Location section of the Edit Page screen is also affected.

As this is a major blocker for us, I've spent a little time looking at the script. How about using asynchronous timers instead of loops to load and update? jQuery's Treeview seems to work quite well by parsing the DOM tree, so another idea would be preloading to a hidden container.

[ Permlink | « Hide ]

Agnes Ro [Atlassian] added a comment - 19/Nov/08 12:41 AM

Fixed for 2.10 rc1.

[ Permlink | « Hide ]

Agnes Ro [Atlassian] added a comment - 19/Nov/08 12:46 AM

We have fixed this so that it no longer hangs. However, due to IE slowness (it's js engine is much slower than other browsers) it is still quite sluggish with 1000+ nodes when doing a drag/drop.

[ Permlink | « Hide ]

Andrew Prentice added a comment - 27/Nov/08 09:31 PM - edited

Reopening because Safari 3.1.2 hangs when viewing tree view irrespective of number of pages in space

verified fixed in 2.10 rc1 for FF2, FF3, IE6, IE7 when viewing space with 1000 pages

[ Permlink | « Hide ]

Agnes Ro [Atlassian] added a comment - 30/Nov/08 10:51 PM

Safari bug fixed for 2.10 rc2

[ Permlink | « Hide ]

Mark Hrynczak [Atlassian] added a comment - 01/Dec/08 07:15 PM

Tree view now works as expected on Safari 3.1.2