Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #16243 from Snuffleupagus/toolbars-object-spread

Browse Source 
Use object-spread when dispatching events in the toolbars
This commit is contained in:
Tim van der Meij 2023-04-02 13:49:03 +02:00 committed by GitHub
commit 1c7e2defea
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
web/secondary_toolbar.js
View File

@ -203,11 +203,7 @@ class SecondaryToolbar {
for (const { element, eventName, close, eventDetails } of this.buttons) {
element.addEventListener("click", evt => {
if (eventName !== null) {
const details = { source: this };
for (const property in eventDetails) {
details[property] = eventDetails[property];
}
this.eventBus.dispatch(eventName, details);
this.eventBus.dispatch(eventName, { source: this, ...eventDetails });
}
if (close) {
this.close();

8
web/toolbar.js
View File

@ -150,13 +150,7 @@ class Toolbar {
for (const { element, eventName, eventDetails } of this.buttons) {
element.addEventListener("click", evt => {
if (eventName !== null) {
const details = { source: this };
if (eventDetails) {
for (const property in eventDetails) {
details[property] = eventDetails[property];
}
}
this.eventBus.dispatch(eventName, details);
this.eventBus.dispatch(eventName, { source: this, ...eventDetails });
}
});
}