diff options
author | toasted-nutbread <toasted-nutbread@users.noreply.github.com> | 2024-02-20 10:13:57 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-20 15:13:57 +0000 |
commit | fe875bbd99980b175fc366a2bfd4395be9cbad72 (patch) | |
tree | 9eacc919cae0aa32c1a5e25b8493a0e86a21f16e /ext/js/comm | |
parent | 0e4ae922451af967c78616057ed26b85ba5d4b5c (diff) |
Constructor simplification (#713)
* Update AudioDownloader
* Update Translator
* Update ClipboardMonitor
* Update ClipboardReader
* Update PanelElement
* Update QueryParser
* Update DisplayGenerator
* Update DisplayHistory
* Update DOMDataBinder
* Remove unnecessary cast
* Update Popup types
* One declaration per line
* Remove optionals from Frontend constructor
* Fix Translator constructor
Diffstat (limited to 'ext/js/comm')
-rw-r--r-- | ext/js/comm/clipboard-monitor.js | 4 | ||||
-rw-r--r-- | ext/js/comm/clipboard-reader.js | 7 |
2 files changed, 6 insertions, 5 deletions
diff --git a/ext/js/comm/clipboard-monitor.js b/ext/js/comm/clipboard-monitor.js index a7cd8833..d101b467 100644 --- a/ext/js/comm/clipboard-monitor.js +++ b/ext/js/comm/clipboard-monitor.js @@ -24,9 +24,9 @@ import {isStringPartiallyJapanese} from '../language/ja/japanese.js'; */ export class ClipboardMonitor extends EventDispatcher { /** - * @param {{clipboardReader: import('clipboard-monitor').ClipboardReaderLike}} details + * @param {import('clipboard-monitor').ClipboardReaderLike} clipboardReader */ - constructor({clipboardReader}) { + constructor(clipboardReader) { super(); /** @type {import('clipboard-monitor').ClipboardReaderLike} */ this._clipboardReader = clipboardReader; diff --git a/ext/js/comm/clipboard-reader.js b/ext/js/comm/clipboard-reader.js index b040d6ca..c09e9ff2 100644 --- a/ext/js/comm/clipboard-reader.js +++ b/ext/js/comm/clipboard-reader.js @@ -23,10 +23,11 @@ import {getFileExtensionFromImageMediaType} from '../media/media-util.js'; */ export class ClipboardReader { /** - * Creates a new instances of a clipboard reader. - * @param {{document: ?Document, pasteTargetSelector: ?string, richContentPasteTargetSelector: ?string}} details Details about how to set up the instance. + * @param {?Document} document + * @param {?string} pasteTargetSelector + * @param {?string} richContentPasteTargetSelector */ - constructor({document = null, pasteTargetSelector = null, richContentPasteTargetSelector = null}) { + constructor(document, pasteTargetSelector, richContentPasteTargetSelector) { /** @type {?Document} */ this._document = document; /** @type {?import('environment').Browser} */ |