Closed
Bug 660618
Opened 13 years ago
Closed 13 years ago
Don't show progress indicators when using bfcache to go to another page
Categories
(SeaMonkey :: UI Design, enhancement)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philip.chee, Assigned: philip.chee)
References
Details
Attachments
(1 file, 1 obsolete file)
|
3.60 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
From Bug 310738 > When the status bar is enabled, back/forward using bfcache does not work as > fast as when it is disabled.
|
Assignee | |
Comment 1•13 years ago
|
||
I don't see any difference in the UI frankly. Presumably back in 2008 it was more noticeable since the progress was a performance hog.
Attachment #536063 -
Flags: review?(neil)
|
||
Comment 2•13 years ago
|
||
Comment on attachment 536063 [details] [diff] [review] Patch v1.0 >+ if (!(aStateFlags & nsIWebProgressListener.STATE_RESTORING)) { >+ this._busyUI = true; I don't think we need to go to all that trouble, just don't turn on the throbber or the progress meter.
|
|
Assignee | |
Comment 3•13 years ago
|
||
> neil@parkwaycc.co.uk 2011-05-30 06:34:56 PDT
>
>>+ if (!(aStateFlags & nsIWebProgressListener.STATE_RESTORING)) {
>>+ this._busyUI = true;
> I don't think we need to go to all that trouble, just don't turn on the throbber or the progress meter.
Fixed.
Attachment #536063 -
Attachment is obsolete: true
Attachment #536063 -
Flags: review?(neil)
Attachment #536846 -
Flags: review?(neil)
|
|
||
Updated•13 years ago
|
Attachment #536846 -
Flags: review?(neil) → review+
|
|
Assignee | |
Comment 4•13 years ago
|
||
Pushed to comm-central http://hg.mozilla.org/comm-central/rev/00d0a464d61e
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•