Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-09 03:04:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Use caret position from point rahter than range from point as the former is standards compliant

Browse Source
This commit is contained in:
Kovid Goyal 2020-08-10 09:45:56 +05:30
parent 6d0cc73d93
commit 41717d5eca
No known key found for this signature in database
GPG Key ID: 06BC317B515ACE7C
1 changed files with 16 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/pyj/select.pyj
View File

@ -3,17 +3,13 @@
from __python__ import bound_methods, hash_literals
def range_from_point(x, y):
r = None
def caret_position_from_point(x, y):
if document.caretPositionFromPoint:
p = document.caretPositionFromPoint(x, y)
if p:
r = document.createRange()
r.setStart(p.offsetNode, p.offset)
r.collapse(True)
elif document.caretRangeFromPoint:
r = document.caretRangeFromPoint(x, y)
return r
return document.caretPositionFromPoint(x, y)
r = document.caretRangeFromPoint(x, y)
if r:
return {'offsetNode': r.startContainer, 'offset': r.startOffset}
return None
def word_boundary_regex():
@ -35,12 +31,13 @@ def expand_offset_to_word(string, offset):
def word_at_point(x, y):
r = range_from_point(x, y)
if r and r.startContainer.nodeType is 3:
word_info = expand_offset_to_word(r.startContainer.data, r.startOffset)
p = caret_position_from_point(x, y)
if p and p.offsetNode?.nodeType is Node.TEXT_NODE:
word_info = expand_offset_to_word(p.offsetNode.data, p.offset)
if word_info.word:
r.setStart(r.startContainer, word_info.start)
r.setEnd(r.startContainer, word_info.end)
r = document.createRange()
r.setStart(p.offsetNode, word_info.start)
r.setEnd(p.offsetNode, word_info.end)
return r
@ -105,10 +102,10 @@ def move_end_of_selection(pos, start):
sel = window.getSelection()
if not sel.rangeCount:
return
newr = range_from_point(pos.x, pos.y)
if newr:
p = caret_position_from_point(pos.x, pos.y)
if p:
r = sel.getRangeAt(0)
if start:
r.setStart(newr.startContainer, newr.startOffset)
r.setStart(p.offsetNode, p.offset)
else:
r.setEnd(newr.endContainer, newr.endOffset)
r.setEnd(p.offsetNode, p.offset)