Bug 105707

Summary: Web Inspector: [Timeline] Can't open collapsed record while pop-up is visible.
Product: WebKit Reporter: Eugene Klyuchnikov <eustas>
Component: Web Inspector (Deprecated)Assignee: Eugene Klyuchnikov <eustas>
Status: RESOLVED WONTFIX    
Severity: Normal CC: apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Eugene Klyuchnikov
Reported 2012-12-23 22:27:18 PST
Original issue: https://code.google.com/p/chromium/issues/detail?id=166907
Attachments
Patch (5.20 KB, patch)
2012-12-23 23:24 PST, Eugene Klyuchnikov 		no flags
DetailsFormatted DiffDiff
Patch (7.12 KB, patch)
2012-12-28 05:42 PST, Eugene Klyuchnikov 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Eugene Klyuchnikov
Comment 1 2012-12-23 23:24:20 PST
Created attachment 180648 [details] Patch
Pavel Feldman
Comment 2 2012-12-23 23:42:50 PST
Comment on attachment 180648 [details] Patch This is likely to regress something. Please blame the history to find the casebit was covering.
Eugene Klyuchnikov
Comment 3 2012-12-24 00:36:52 PST
(In reply to comment #2) > (From update of attachment 180648 [details]) > This is likely to regress something. Please blame the history to find the casebit was covering. I've looked over all changesets and found the following requirements: * Popover should appear like a tooltip * Popover should not appear after panel has been changed * Color picker should work (choose color) * Object tree inside popover should work * Popover should hide on click outside of popover * Popover should hide when navigate to another panel * Popover should not flicker when pointer moves between anchor representatives * It should be easy to reach popover content All of them are met by this patch. In this patch we only remove hide-out timer that makes logic complicated and replace it with straightforward position check. PTAL. Thanks.
Eugene Klyuchnikov
Comment 4 2012-12-28 05:42:56 PST
Created attachment 180867 [details] Patch
Note You need to log in before you can comment on or make changes to this bug.