Bookhoard/foliate-js
1
0
Fork 0
mirror of https://github.com/john-okeefe/foliate-js.git synced 2026-09-30 23:49:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Keep the long-pressed word covered when Chrome re-anchors a touch drag

Browse Source 
Dragging just past a line-start word makes Chrome flip the touch
selection and re-anchor it at a rendered line break, silently
dropping the long-pressed word from the selection. Record the word
at the initial long-press and restore its start boundary whenever
the selection no longer covers it.
This commit is contained in:
John O'Keefe
2026-09-13 20:16:10 -04:00
parent 800fb2fa02
commit 3fcfc30bd0
1 changed files with 32 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
paginator.js
+32 -1
View File
@@ -608,6 +608,34 @@ export class Paginator extends HTMLElement {
touchSelecting = e.pointerType === 'touch'
})
doc.addEventListener('keydown', () => touchSelecting = false)
// The word selected by the initial long-press. Chrome can
// re-anchor a touch drag at a rendered line break when the
// finger crosses just past a line-start word, silently dropping
// that word from the selection; keep it covered.
let anchorWordStart = null
doc.addEventListener('pointerdown', e => {
if (e.pointerType === 'touch') anchorWordStart = null
})
const preserveAnchorWord = sel => {
if (!sel.rangeCount || sel.type !== 'Range') return
const r = sel.getRangeAt(0)
if (!anchorWordStart) {
anchorWordStart =
{ node: r.startContainer, offset: r.startOffset }
return
}
const probe = doc.createRange()
probe.setStart(anchorWordStart.node, anchorWordStart.offset)
probe.collapse(true)
if (r.compareBoundaryPoints(Range.START_TO_START, probe) <= 0)
return
if (selectionIsBackward(sel))
sel.setBaseAndExtent(sel.anchorNode, sel.anchorOffset,
probe.startContainer, probe.startOffset)
else
sel.setBaseAndExtent(probe.startContainer, probe.startOffset,
sel.focusNode, sel.focusOffset)
}
let isKeyboardSelecting = false
doc.addEventListener('keydown', () => isKeyboardSelecting = true)
doc.addEventListener('keyup', () => isKeyboardSelecting = false)
@@ -618,6 +646,7 @@ export class Paginator extends HTMLElement {
const sel = doc.getSelection()
if (!sel.rangeCount) return
if (touchSelecting && sel.type === 'Range') {
preserveAnchorWord(sel)
this.#clampTouchSelection(sel, doc)
// Chrome ignores (or re-maps) selection writes made from
// JS while the touch selection gesture is active, and no
@@ -627,8 +656,10 @@ export class Paginator extends HTMLElement {
// gesture has most likely ended.
const clampLater = () => {
const s = doc.getSelection()
if (s && s.rangeCount && s.type === 'Range')
if (s && s.rangeCount && s.type === 'Range') {
preserveAnchorWord(s)
this.#clampTouchSelection(s, doc)
}
}
setTimeout(clampLater, 150)
setTimeout(clampLater, 500)