Skip to content

Commit ed9d8aa

Browse files
committed
remove old hack that had backspace as keybinding for delete option
1 parent b9e7c5e commit ed9d8aa

2 files changed

Lines changed: 2 additions & 9 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ _Breaking developer changes, which may affect downstream projects or sites that
4444
#### :white_check_mark: Validation
4545
#### :bug: Bugfixes
4646
* Restore dedicated rendering of ski pistes and building parts ([#12297], thanks [@matkoniecz])
47+
* Pressing backspace while in the feature type selecting mode should not delete the object
4748
#### :earth_asia: Localization
4849
#### :hourglass: Performance
4950
#### :mortar_board: Walkthrough / Help

modules/ui/preset_list.js

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -45,16 +45,8 @@ export function uiPresetList(context) {
4545
.call(svgIcon('#iD-icon-close'));
4646

4747
function initialKeydown(d3_event) {
48-
// hack to let delete shortcut work when search is autofocused
49-
if (search.property('value').length === 0 &&
50-
(d3_event.keyCode === utilKeybinding.keyCodes['⌫'] ||
51-
d3_event.keyCode === utilKeybinding.keyCodes['⌦'])) {
52-
d3_event.preventDefault();
53-
d3_event.stopPropagation();
54-
operationDelete(context, _entityIDs)();
55-
5648
// hack to let undo work when search is autofocused
57-
} else if (search.property('value').length === 0 &&
49+
if (search.property('value').length === 0 &&
5850
(d3_event.ctrlKey || d3_event.metaKey) &&
5951
d3_event.keyCode === utilKeybinding.keyCodes.z) {
6052
d3_event.preventDefault();

0 commit comments

Comments
 (0)