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

Merge pull request #17388 from calixteman/no_double_color_picker

Browse Source 
[Editor] Avoid to have a color picker for highlighting twice in the main toolbar
This commit is contained in:
calixteman 2023-12-06 16:29:33 +01:00 committed by GitHub
commit f54cfe065a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
web/toolbar.js
View File

@ -121,12 +121,17 @@ class Toolbar {
this.#bindListeners(options); this.#bindListeners(options);
if (options.editorHighlightColorPicker) { if (options.editorHighlightColorPicker) {
this.eventBus._on("annotationeditoruimanager", ({ uiManager }) => { this.eventBus._on(
this.#setAnnotationEditorUIManager( "annotationeditoruimanager",
uiManager, ({ uiManager }) => {
options.editorHighlightColorPicker this.#setAnnotationEditorUIManager(
); uiManager,
}); options.editorHighlightColorPicker
);
},
// Once the color picker has been added, we don't want to add it again.
{ once: true }
);
} }
this.reset(); this.reset();