Skip to content

Commit

Permalink
Revert "editing: Added command and keybindings to remove previously s…
Browse files Browse the repository at this point in the history
…elected word in multi-caret session - fixes #869"

This reverts commit e050896.
  • Loading branch information
mitchell-as committed Dec 1, 2015
1 parent b234fbf commit ee5b7eb
Show file tree
Hide file tree
Showing 8 changed files with 4 additions and 55 deletions.
5 changes: 0 additions & 5 deletions src/chrome/komodo/content/commandsOverlay.p.xul
Expand Up @@ -1567,11 +1567,6 @@
disabled="true"
desc="&generalPrintPreviewSelection.desc;"/>
<!-- #endif -->
<command
id="cmd_removePrevWordInCaretSet" key="key_cmd_removePrevWordInCaretSet"
oncommand="ko.commands.doCommandAsync('cmd_removePrevWordInCaretSet', event)"
disabled="true"
desc="&removePrevWordInCaretSet.desc;"/>
</commandset>

<!-- These commands need to be updated on window open,
Expand Down
27 changes: 0 additions & 27 deletions src/chrome/komodo/content/editor.js
Expand Up @@ -584,33 +584,6 @@ editor_editorController.prototype.do_cmd_addNextWordToCaretSet = function() {
scimoz.searchFlags |= scimoz.SCFIND_WHOLEWORD;
}
scimoz.multipleSelectAddNext();
scimoz.sendUpdateCommands("select");
};

var is_cmd_removeAdditionalCaret_enabled_aux = function() {
var view = _getCurrentScimozView();
if (!view) {
return null;
}
var scimoz = view.scimoz;
if (!scimoz) {
return null;
}
return (scimoz.selections > 1) ? view : null;
};

editor_editorController.prototype.is_cmd_removePrevWordInCaretSet_enabled = function() {
return !!is_cmd_removeAdditionalCaret_enabled_aux();
}

editor_editorController.prototype.do_cmd_removePrevWordInCaretSet = function() {
var view = is_cmd_removeAdditionalCaret_enabled_aux();
if (!view) {
return;
}
var scimoz = view.scimoz;
scimoz.dropSelectionN(scimoz.selections - 1);
scimoz.sendUpdateCommands("select");
};

editor_editorController.prototype._aux_is_cmd_rename_tag_enabled = function() {
Expand Down
8 changes: 1 addition & 7 deletions src/chrome/komodo/content/keybindings/keybindings.p.js
Expand Up @@ -331,7 +331,6 @@ function cloneObject(what) {
* existing keybindings to move to the state you're implementing for N+1
*
* Version history:
* 49: Komodo 10.0 - add Ctrl+Shift+D for cmd_removePrevWordInCaretSet
* 48: Komodo 9.3 - Ctrl+T is now used for new tabs
* 47: Komodo 9.3 - Add keybindings for quick bookmarks
* Juggle help (and friends) and commenting keybindings to
Expand Down Expand Up @@ -381,7 +380,7 @@ function cloneObject(what) {
* 2: Komodo 4.2.0-beta2 and above
* 1: Komodo 4.2.0-beta1 and before
*/
const currentKeybindingVersionNumber = 49;
const currentKeybindingVersionNumber = 48;

/**
* Remove this dictionary of keybinds.
Expand Down Expand Up @@ -1064,11 +1063,6 @@ this.Manager.prototype._upgradeKeybingings = function (from_version,
});
// #endif
break;
case 49:
this._add_keybinding_sequences({
'cmd_removePrevWordInCaretSet': ["Ctrl+Shift+D"],
});
break;
}
from_version += 1;
}
Expand Down
7 changes: 0 additions & 7 deletions src/chrome/komodo/content/komodo.p.xul
Expand Up @@ -278,9 +278,6 @@
<menuitem id="editor-context-addNextWordToCaretSet"
label="&addNextWordToCaretSet.label;"
observes="cmd_addNextWordToCaretSet"/>
<menuitem id="editor-context-removePrevWordInCaretSet"
label="&removePrevWordInCaretSet.label;"
observes="cmd_removePrevWordInCaretSet"/>
</menupopup>
</menu>
<menuitem id="editor-context-find"
Expand Down Expand Up @@ -858,10 +855,6 @@
id="menu_addNextWordToCaretSet"
observes="cmd_addNextWordToCaretSet"
/>
<menuitem label="&removePrevWordInCaretSet.label;"
id="menu_removePrevWordInCaretSet"
observes="cmd_removePrevWordInCaretSet"
/>
</menupopup>
</menu>

Expand Down
2 changes: 0 additions & 2 deletions src/chrome/komodo/locale/en-US/komodo.dtd
Expand Up @@ -603,8 +603,6 @@
<!ENTITY quitApplicationCmdMac.accesskey "Q">
<!ENTITY quitApplicationCmdWin.label "Exit">
<!ENTITY quitApplicationCmdWin.accesskey "x">
<!ENTITY removePrevWordInCaretSet.label "Multiple Selection - Remove Previous Occurrence">
<!ENTITY removePrevWordInCaretSet.desc "Editor: Remove Previous Word in a Multi-Caret Set">
<!ENTITY rename.label "Rename...">
<!ENTITY renameTag.label "Rename Tag">
<!ENTITY renameTag.desc "Editor: Rename Start- and End-Tag Name Simultaneously">
Expand Down
4 changes: 1 addition & 3 deletions src/schemes/Default.p.kkf
@@ -1,4 +1,4 @@
version 49
version 48

# Movement and selection
binding cmd_left Left
Expand Down Expand Up @@ -224,5 +224,3 @@ binding cmd_scope-openfiles Ctrl+<
binding cmd_invokeHyperlink Ctrl+Shift+G

binding cmd_addNextWordToCaretSet Ctrl+D
binding cmd_removePrevWordInCaretSet Ctrl+Shift+D

3 changes: 1 addition & 2 deletions src/schemes/Emacs.p.kkf
@@ -1,4 +1,4 @@
version 49
version 48

# Needs a few things to be finished
# - ?? insert forward tab / insert backward tab
Expand Down Expand Up @@ -231,4 +231,3 @@ binding cmd_scope-openfiles Ctrl+<
binding cmd_invokeHyperlink Ctrl+Shift+G

#binding cmd_addNextWordToCaretSet Ctrl+K Ctrl+D
#binding cmd_removePrevWordInCaretSet Ctrl+K Ctrl+D
3 changes: 1 addition & 2 deletions src/schemes/Mac.p.kkf
@@ -1,4 +1,4 @@
version 49
version 48

binding cmd_back Shift+Backspace
binding cmd_backSmart Backspace
Expand Down Expand Up @@ -193,7 +193,6 @@ binding cmd_scope-openfiles Meta+<
binding cmd_invokeHyperlink Ctrl+Shift+G

binding cmd_addNextWordToCaretSet Ctrl+D
binding cmd_removePrevWordInCaretSet Ctrl+Shift+D
binding cmd_htmlTagRelocator Ctrl+Shift+Z

binding cmd_lineTransposeUp Ctrl+Up
Expand Down

0 comments on commit ee5b7eb

Please sign in to comment.