Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Slows down IE 10 considerably #25

Closed
alexzeitgeist opened this issue Oct 22, 2013 · 2 comments
Closed

Slows down IE 10 considerably #25

alexzeitgeist opened this issue Oct 22, 2013 · 2 comments

Comments

@alexzeitgeist
Copy link

Run Roundcube with and without Contextmenu inside IE 10. Start the IE Profiler. If you switch to current view: Call tree, you will notice that with Contextmenu enabled, the total time for rendering the page view goes up by almost 100%.

Without Contextmenu:
without_contextmenu

With Contextmenu:
with_contextmenu

The code that is likely responsible for the delay:

    if (window.rcmail) {
        // init message list menu
        if ($('#rcmContextMenu').length > 0) {
            rcmail.addEventListener('listupdate', function(props) { rcm_contextmenu_update(); } );
            rcmail.addEventListener('insertrow', function(props) { rcm_contextmenu_init(props.row.id); } ); // <--------- slow down
        }

Note that I have 50 rows displayed in my mail view by default. So that function rcmail.addEventListener('insertrow'....) gets called 50 times. The more rows you display, the more noticeable the relative slowdown compared to running RC without contextmenu.

@johndoh
Copy link
Owner

johndoh commented Feb 24, 2014

I am working on a new version in the devel-v2 branch. This is much faster, you might be interested to try it (its only dev so far and requires Roundcube v1.0-rc or git-master).

@johndoh
Copy link
Owner

johndoh commented Apr 30, 2014

please try with release 2.0 and reopen if there is still a problem.

@johndoh johndoh closed this as completed Apr 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants