Bug 67021 - Web Inspector: assign CtrlOrMeta+s shortcut to saveAs in ScriptsPanel.
Summary: Web Inspector: assign CtrlOrMeta+s shortcut to saveAs in ScriptsPanel.
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Roland Takacs
URL:
Keywords:
Depends on:
Blocks: 116924
  Show dependency tree / graph
 
Reported: 2011-08-25 23:31 PDT by Ilya Tikhonovsky
Modified: 2014-05-27 04:07 PDT (History)
13 users (show)

See Also:

Attachments
[patch] initial version (2.00 KB, patch)
2011-08-25 23:37 PDT, Ilya Tikhonovsky 		pfeldman: review-
Details | Formatted Diff | Diff
Patch (2.25 KB, patch)
2013-06-06 05:41 PDT, Mátyás Mustoha 		ossy: review-
ossy: commit-queue-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ilya Tikhonovsky 2011-08-25 23:31:39 PDT 
%subj%
Comment 1 Ilya Tikhonovsky 2011-08-25 23:37:44 PDT 
Created attachment 105317 [details]
[patch] initial version
Comment 2 Pavel Feldman 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.
Comment 3 Roland Takacs 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.
Comment 4 Mátyás Mustoha 2013-06-06 05:41:49 PDT 
Created attachment 203925 [details]
Patch
Comment 5 Noam Rosenthal 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?
Comment 6 Mátyás Mustoha 2013-06-10 02:41:22 PDT 
Yes, it should open the dialog (except when auto-download is set in the browser).
Comment 7 Csaba Osztrogonác 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?
Comment 8 Csaba Osztrogonác 2014-05-27 04:07:42 PDT 
It is a bug for the old inspector UI.