Skip to content

Commit 24d0cb5

Browse files
committed
changes whilre reviewing
1 parent 3c698a3 commit 24d0cb5

2 files changed

Lines changed: 12 additions & 10 deletions

File tree

components/annotorious-annotator/line-parser.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -288,12 +288,11 @@ class AnnotoriousAnnotator extends HTMLElement {
288288
tpen-page-selector::part(select) {
289289
font-size: clamp(0.8rem, 1vw, 1rem);
290290
padding: 10px 15px;
291-
border: none;
291+
border: 2px outset buttonborder;
292292
border-radius: 5px;
293293
background-color: var(--primary-color);
294294
color: var(--white);
295295
cursor: pointer;
296-
border: 2px outset buttonborder;
297296
}
298297
tpen-page-selector::part(select):hover,
299298
tpen-page-selector::part(select):focus-visible {

components/page-selector/index.js

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ import CheckPermissions from "../check-permissions/checkPermissions.js"
33
import { onProjectReady } from "../../utilities/projectReady.js"
44
import { CleanupRegistry } from '../../utilities/CleanupRegistry.js'
55

6-
const eventDispatcher = TPEN.eventDispatcher
7-
86
/**
97
* PageSelector - Dropdown for selecting pages within a project.
108
* Provides navigation between manuscript pages.
@@ -64,14 +62,19 @@ export default class PageSelector extends HTMLElement {
6462
#buildPagesList() {
6563
this.#pages = []
6664
const layers = TPEN.activeProject?.layers || []
65+
const seenIds = new Set()
6766

6867
layers.forEach(layer => {
6968
(layer.pages || []).forEach(page => {
70-
this.#pages.push({
71-
id: page.id.split('/').pop(),
72-
fullId: page.id,
73-
label: this.#getLabel(page)
74-
})
69+
const id = page.id.split('/').pop()
70+
if (!seenIds.has(id)) {
71+
seenIds.add(id)
72+
this.#pages.push({
73+
id: id,
74+
fullId: page.id,
75+
label: this.#getLabel(page)
76+
})
77+
}
7578
})
7679
})
7780

@@ -145,7 +148,7 @@ export default class PageSelector extends HTMLElement {
145148
const selectedPage = this.#pages.find(p => p.id === selectedPageId)
146149

147150
// Dispatch event for interested components
148-
eventDispatcher.dispatch('tpen-page-selected', {
151+
TPEN.eventDispatcher.dispatch('tpen-page-selected', {
149152
pageId: selectedPageId,
150153
pageIndex: this.#pages.indexOf(selectedPage),
151154
page: selectedPage

0 commit comments

Comments
 (0)