Opened 18 months ago

Closed 18 months ago

Last modified 18 months ago

#1488207 closed Bugs (fixed)

fit_string_to_size renders browser and ui unresponsive

Reported by: maharaja Owned by:
Priority: 5 Milestone: 0.7-stable
Component: User Interface Version: 0.6
Severity: normal Keywords:
Cc:

Description

hi!

per our discussion on the mailing list i open a ticket for fit_string_to_size() in skins/default/functions.js .

when displaying a lot of folders (currently nearly 2000 folders in one of my mailboxes) the browser/the ui gets unresponsive. after a certain timeout, the browser (firefox 9 beta in my case) prompts the user if he wants to stop the script in fit_string_to_size().

alec confirmed, that fit_string_to_size() can be very expensive.

i'm in no way familiar with javascript in regards to the ui but maybe there is a possibility to cache the fit_string_to_size output as long as the folder pane does not change in size?

thanks,
raoul

Change History (3)

comment:1 Changed 18 months ago by alec

Just to be sure did you test with all plugins disabled?

comment:2 Changed 18 months ago by alec

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

Done in [79db3309]: 1) improve its performance by half, 2) don't call it on UI init, it's called after getunread action 3) don't call it for big number of folders (limit is 100 for IE and 500 for others).

comment:3 Changed 18 months ago by maharaja

hi!

i can confirm that this issue has been addressed and things work as expected!

thank you,
raoul

Note: See TracTickets for help on using tickets.