Summary: | Web Inspector: panel switching is too slow | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ilya Tikhonovsky <loislo> | ||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Ilya Tikhonovsky <loislo> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 61557, 65017 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Ilya Tikhonovsky
2011-05-17 07:58:48 PDT
Created attachment 93765 [details]
[patch] initial version
Comment on attachment 93765 [details]
[patch] initial version
Did you try visibility:hidden rather than opacity? It might be faster
(In reply to comment #2) > (From update of attachment 93765 [details]) > Did you try visibility:hidden rather than opacity? It might be faster I've checked this right now and get the next values My test case: Inspector opened for gmail page with expanded body children items I'm switching inspector from resource panel to elements panel and back display:none r2e ~550ms, e2r ~8.5sec visibility: hidden: r2e ~340ms, e2r ~125ms opacity: 0: r2e ~150ms, e2r ~37ms Comment on attachment 93765 [details]
[patch] initial version
As discussed offline clearing r? while Ilya is looking for a better solution.
Created attachment 100507 [details]
[patch] second version
Looks like we have no better solution for panel switching.
Created attachment 100772 [details]
[patch] third version
there was a problem with mouse events.
It happened with scripts panel when it is opened after profile panel.
Comment on attachment 100772 [details]
[patch] third version
Can we "inherit" all views from ".view" and specify .visible once? Panel inherits from View as well, so it'll work for it.
Comment on attachment 93765 [details]
[patch] initial version
After offline discussion it was decided to use opacity only for panels. It is not possible to use such solution for all the views because in that case we need to manage the visible state of each the view explicitly.
Comment on attachment 93765 [details] [patch] initial version Clearing flags on attachment: 93765 Committed r91024: <http://trac.webkit.org/changeset/91024> All reviewed patches have been landed. Closing bug. |