diff options
author | MCMXC <16797721+mcmxcdev@users.noreply.github.com> | 2023-12-07 22:37:37 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-07 22:37:37 -0700 |
commit | a774c879d4cf08a76fc404771883807d27465265 (patch) | |
tree | 202c9afdeb4830d417e0ec7137a01e7981c8651c /src/webview | |
parent | 6.6.1-nightly.17 [skip ci] (diff) | |
download | ferdium-app-a774c879d4cf08a76fc404771883807d27465265.tar.gz ferdium-app-a774c879d4cf08a76fc404771883807d27465265.tar.zst ferdium-app-a774c879d4cf08a76fc404771883807d27465265.zip |
refactor: code cleanup (#1476)
- auto update `settings.json` due to new vscode version
- replace deprecated `e.keyCode` with `e.key`
- fix various code snippets with sonarlint and other lint plugins
Diffstat (limited to 'src/webview')
-rw-r--r-- | src/webview/contextMenuBuilder.ts | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/webview/contextMenuBuilder.ts b/src/webview/contextMenuBuilder.ts index 93dfbebb6..ee55fbe62 100644 --- a/src/webview/contextMenuBuilder.ts +++ b/src/webview/contextMenuBuilder.ts | |||
@@ -101,7 +101,7 @@ function translatePopup(res, isError: boolean = false) { | |||
101 | 101 | ||
102 | document.addEventListener('click', e => { | 102 | document.addEventListener('click', e => { |
103 | if (div !== e.target && !childOf(e.target, div)) { | 103 | if (div !== e.target && !childOf(e.target, div)) { |
104 | div?.remove(); | 104 | div.remove(); |
105 | } | 105 | } |
106 | }); | 106 | }); |
107 | } | 107 | } |
@@ -253,10 +253,7 @@ export class ContextMenuBuilder { | |||
253 | return this.buildMenuForVideo(info); | 253 | return this.buildMenuForVideo(info); |
254 | } | 254 | } |
255 | 255 | ||
256 | if ( | 256 | if (info.isEditable || info.inputFieldType !== 'none') { |
257 | info.isEditable || | ||
258 | (info.inputFieldType && info.inputFieldType !== 'none') | ||
259 | ) { | ||
260 | return this.buildMenuForTextInput(info); | 257 | return this.buildMenuForTextInput(info); |
261 | } | 258 | } |
262 | 259 | ||