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

Description Vsevolod Vlasov 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).
Comment 1 Vsevolod Vlasov 2012-01-24 22:59:49 PST
 - Added close button to search panel.
 - Esc now closes the panel even when search filed has focus.
Comment 2 Vsevolod Vlasov 2012-01-24 23:00:31 PST
Created attachment 123887 [details]
Patch
Comment 3 Pavel Feldman 2012-01-25 01:42:18 PST
Is there a screenshot?
Comment 4 Vsevolod Vlasov 2012-01-25 02:18:52 PST
Created attachment 123902 [details]
Screenshot
Comment 5 Pavel Feldman 2012-01-25 04:09:27 PST
Comment on attachment 123887 [details]
Patch

Please make sure this layout works fine on a Mac.
Comment 6 Vsevolod Vlasov 2012-01-26 00:16:23 PST
Committed r105969: <http://trac.webkit.org/changeset/105969>