Bug 67021

Summary: Web Inspector: assign CtrlOrMeta+s shortcut to saveAs in ScriptsPanel.
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Roland Takacs <rtakacs>
Status: RESOLVED INVALID    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, mmatyas, noam, ossy, pfeldman, pmuellr, rik, rtakacs, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 116924    
Attachments:
Description Flags
[patch] initial version
pfeldman: review-
Patch ossy: review-, ossy: commit-queue-

Ilya Tikhonovsky
Reported 2011-08-25 23:31:39 PDT
%subj%
Attachments
[patch] initial version (2.00 KB, patch)
2011-08-25 23:37 PDT, Ilya Tikhonovsky 		pfeldman: review-
DetailsFormatted DiffDiff
Patch (2.25 KB, patch)
2013-06-06 05:41 PDT, Mátyás Mustoha 		ossy: review-
ossy: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2011-08-25 23:37:44 PDT
Created attachment 105317 [details] [patch] initial version
Pavel Feldman
Comment 2 2011-08-25 23:46:50 PDT
Comment on attachment 105317 [details] [patch] initial version We should not show dialog upon each commit into the VM. Either there should be a way to commit without saving or Save should be done using different shortcut.
Roland Takacs
Comment 3 2013-05-30 06:02:49 PDT
Since there is no activity on the bug for a long time I had the courage to take over the bug.
Mátyás Mustoha
Comment 4 2013-06-06 05:41:49 PDT
Created attachment 203925 [details] Patch
Noam Rosenthal
Comment 5 2013-06-10 02:36:36 PDT
Comment on attachment 203925 [details] Patch Does this actually open a "Save as" dialog? or does it just save the script with a constructed filename?
Mátyás Mustoha
Comment 6 2013-06-10 02:41:22 PDT
Yes, it should open the dialog (except when auto-download is set in the browser).
Csaba Osztrogonác
Comment 7 2013-10-21 03:51:22 PDT
Comment on attachment 203925 [details] Patch r-, because old inspector UI was removed, so it is obsolete. Do we have this feature in the new inspector?
Csaba Osztrogonác
Comment 8 2014-05-27 04:07:42 PDT
It is a bug for the old inspector UI.
Note You need to log in before you can comment on or make changes to this bug.