Closed
Bug 84676
Opened 23 years ago
Closed 23 years ago
Tabbing from address widget performance bites!
Categories
(MailNews Core :: Composition, defect, P1)
Tracking
(Not tracked)
mozilla0.9.4
People
(Reporter: selmer, Assigned: bugzilla)
References
Details
(Keywords: perf, Whiteboard: [nsbeta1+])
6/7 13 branch win32
I'm not sure when this started, it was a while ago. It seems to keep getting worse!
Create a new compose window
Wait around to type your address
Complete any autocompletion so you don't hit the bug where tab selects addresses
Hit tab once or twice depending on whether you'd like to wait to enter the
subject or the body
Start typing
Wait for any characters to show up
I can generally type 2 tabs and
Jean-Francois,
Why so slow?
before seeing any text in the message body.
Reporter | ||
Comment 1•23 years ago
|
||
We should look at whether there's an obvious fix for this before RTM that could
go in a limbo build. There is a build back in m.9 that is not as bad as current
builds. It may be that finding the day this regressed so bad would help figure
out the cause.
I remember thinking that the time it takes to draw focus outlines for the
subject line might be related. Perhaps that can help bound the time period
where this changed.
Comment 2•23 years ago
|
||
I can see two things that could be the cause:
1) The new autocomplete widget
2) LDAP autocomplete
Steve, are you using LDAP?
Yes, this has been really bad, I also see this when typing the @ character, it
hangs for a second or so (no LDAP here), this is slower than normal for me, too.
Comment 5•23 years ago
|
||
cc'ing hewitt
Comment 6•23 years ago
|
||
putting into 0.9.2
Priority: -- → P1
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.2
Comment 7•23 years ago
|
||
Everytime I set the focus in one of the addressing widget edit field, the user
is blocked for about 1-2 seconds!!
Updated•23 years ago
|
Assignee: ducarroz → varada
Comment 8•23 years ago
|
||
reassign to varada
Comment 10•23 years ago
|
||
Is anyone still seeing this? I haven't seen this for a while.
Scott, Varada and I looked at this on my machine, and I'm no longer seeing it
with the trunk builds on Win2K.
Reporter | ||
Comment 12•23 years ago
|
||
Here's another possible clue. Between all address inputs the chrome flashes.
My guess would be command updaters are doing something. It's interesting that
I'm inputting addresses but the spell checker icon will blink even though it's
disabled. Is it possible we're cycling it from disabled to enabled and back in
between address inputs?
Assignee | ||
Comment 13•23 years ago
|
||
Yes, selmer is correct: we're needlessly updating a whole bunch of commands
every time *anything* takes focus in the msg compose window, even if you're not
touching the msg body. I'm working on a fix for that.
Meanwhile, I have a fix that makes focusing the body instanteous after the first
time (and speeds up the first time). It's pretty simple, if this is still being
considered for branch...
Comment 14•23 years ago
|
||
The general perf suckiness when tabbing between widgets in message compose is
now bug 89624. It sounds like some people were having especially bad problems
tabbing from the address widget, so I'm leaving this bug open for now.
Depends on: 89624
Assignee | ||
Comment 15•23 years ago
|
||
Doesn't look like this is getting fixed before the freeze tonight.
Pushing out a milestone. Please correct if I'm mistaken.
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Assignee | ||
Comment 17•23 years ago
|
||
Steve, my patch in bug 89624 should have improved this and window opening time
significantly. I am still seeing a problem where, in certain cases, it takes a
long time to show the caret in the message body (although you can type), the
first time you focus it. Subsquent times are instant though. Please test and
let me know the results, thanks.
*** This bug has been marked as a duplicate of 89624 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
verified dup. Blake has purposely left open bug 89624 to address future
optimizations. As it stands this has improved tremendously. Thanks, Blake.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•