Closed
Bug 1490946
Opened 7 years ago
Closed 2 years ago
Modal dialogs in General Preferences show up too slow
Categories
(Firefox :: Settings UI, defect, P3)
Firefox
Settings UI
Tracking
()
RESOLVED
WORKSFORME
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | affected |
People
(Reporter: mikedeboer, Unassigned)
Details
(Keywords: perf:frontend)
I noticed a performance problem that most likely has to do with populating the controls with data before the dialog is visible.
I think it'd be good to show an empty dialog with the bottom row buttons at the appropriate size and make its contents visible when it's finished loading data.
I noticed considerable lag using both 'Fonts & Colors' dialogs and the one for 'Language'.
Comment 1•7 years ago
|
||
Can you run the profiler to find what is taking so long? We should try to fix the performance issue first.
Flags: needinfo?(mdeboer)
Updated•7 years ago
|
Whiteboard: [fxperf]
Comment 2•7 years ago
|
||
This profile shows a 254ms lag between clicking the "Choose" button in the Languages section, and the dialog appearing.
https://perfht.ml/2O4qqBP
Priority: -- → P3
Whiteboard: [fxperf] → [fxperf:p3]
Comment 3•7 years ago
|
||
Keeping the needinfo on mikedeboer in case the performance he's seeing is worse - in which case, a profile from him would still be needed.
Comment 4•7 years ago
|
||
Interestingly, we also seem to do a full page paint for some reason just before the dialog comes up.
| Reporter | ||
Comment 5•6 years ago
|
||
Thanks for that, Mike, I don't see worse performance than you were able to measure, I think. What you mentioned in comment 4 sounds like something worth looking into ;-)
Flags: needinfo?(mdeboer)
Updated•3 years ago
|
Severity: normal → S3
Comment 6•2 years ago
|
||
I can't reproduce jank anymore when opening the subdialog, but profiling this made me notice the fonts dialog flickers quite a bit. I filed bug 1889445.
Status: NEW → RESOLVED
Closed: 2 years ago
Keywords: perf:frontend
Resolution: --- → WORKSFORME
Whiteboard: [fxperf:p3]
You need to log in
before you can comment on or make changes to this bug.
Description
•