Bug 76983

Summary: Web Inspector: Improve user experience of advanced search panel closing.
Product: WebKit Reporter: Vsevolod Vlasov <vsevik>
Component: Web Inspector (Deprecated)Assignee: Vsevolod Vlasov <vsevik>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
pfeldman: review+
Screenshot none

Vsevolod Vlasov
Reported 2012-01-24 22:54:16 PST
Currently the only way to close the panel is to press esc. This is confusing for users. Also esc does not close this panel when search filed has focus (input with type=search clears on esc by default).
Attachments
Patch (6.09 KB, patch)
2012-01-24 23:00 PST, Vsevolod Vlasov 		pfeldman: review+
DetailsFormatted DiffDiff
Screenshot (207.71 KB, image/png)
2012-01-25 02:18 PST, Vsevolod Vlasov 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Vsevolod Vlasov
Comment 1 2012-01-24 22:59:49 PST
- Added close button to search panel. - Esc now closes the panel even when search filed has focus.
Vsevolod Vlasov
Comment 2 2012-01-24 23:00:31 PST
Created attachment 123887 [details] Patch
Pavel Feldman
Comment 3 2012-01-25 01:42:18 PST
Is there a screenshot?
Vsevolod Vlasov
Comment 4 2012-01-25 02:18:52 PST
Created attachment 123902 [details] Screenshot
Pavel Feldman
Comment 5 2012-01-25 04:09:27 PST
Comment on attachment 123887 [details] Patch Please make sure this layout works fine on a Mac.
Vsevolod Vlasov
Comment 6 2012-01-26 00:16:23 PST
Committed r105969: <http://trac.webkit.org/changeset/105969>
Note You need to log in before you can comment on or make changes to this bug.