alex/yomichan
1
Fork 0
Code Releases Activity

Update hotkey issues (#1244)

Browse Source 
* Scroll to the bottom when adding inputs

* Skip if empty action
This commit is contained in:
toasted-nutbread 2021-01-15 21:19:22 -05:00 committed by GitHub
parent 1c5e53a7ac
commit dc4d659184
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ext/bg/js/settings2/keyboard-shortcuts-controller.js
View File

@ -30,6 +30,7 @@ class KeyboardShortcutController {
this._listContainer = null;
this._emptyIndicator = null;
this._stringComparer = new Intl.Collator('en-US'); // Invariant locale
this._scrollContainer = null;
}
get settingsController() {
@ -44,6 +45,7 @@ class KeyboardShortcutController {
this._resetButton = document.querySelector('#hotkey-list-reset');
this._listContainer = document.querySelector('#hotkey-list');
this._emptyIndicator = document.querySelector('#hotkey-list-empty');
this._scrollContainer = document.querySelector('#keyboard-shortcuts .modal-body');
this._addButton.addEventListener('click', this._onAddClick.bind(this));
this._resetButton.addEventListener('click', this._onResetClick.bind(this));
@ -65,6 +67,7 @@ class KeyboardShortcutController {
}]);
await this._updateOptions();
this._scrollContainer.scrollTop = this._scrollContainer.scrollHeight;
}
async deleteEntry(index) {

2
ext/mixed/js/display.js
View File

@ -1927,7 +1927,7 @@ class Display extends EventDispatcher {
this._hotkeys.clear();
for (const {action, key, modifiers, scopes, enabled} of options.inputs.hotkeys) {
if (!enabled || !scopes.includes(this._pageType)) { continue; }
if (!enabled || action === '' || !scopes.includes(this._pageType)) { continue; }
this._registerHotkey(key, modifiers, action);
}
}