From fe875bbd99980b175fc366a2bfd4395be9cbad72 Mon Sep 17 00:00:00 2001
From: toasted-nutbread <toasted-nutbread@users.noreply.github.com>
Date: Tue, 20 Feb 2024 10:13:57 -0500
Subject: 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
---
 ext/js/comm/clipboard-monitor.js | 4 ++--
 ext/js/comm/clipboard-reader.js  | 7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)

(limited to 'ext/js/comm')

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} */
-- 
cgit v1.2.3