Skip to content

Dictionary: fix invalid highlight when switching dictionary#14028

Merged
Frenzie merged 1 commit into
koreader:masterfrom
TnS-hun:dictionary-highlight-fix
Jul 6, 2025
Merged

Dictionary: fix invalid highlight when switching dictionary#14028
Frenzie merged 1 commit into
koreader:masterfrom
TnS-hun:dictionary-highlight-fix

Conversation

@TnS-hun

@TnS-hun TnS-hun commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

When the contents of the HtmlBoxWidget changed the page boxes were not invalidated, this resulted the highlight to show up at a wrong coordinate.

Fixes #14013


This change is Reviewable

When the contents of the HtmlBoxWidget changed the page boxes were not invalidated, this resulted the highlight to show up at a wrong coordinate.

Fixes koreader#14013
@Frenzie Frenzie added this to the 2025.06 milestone Jul 6, 2025
@Frenzie Frenzie merged commit 5d3c717 into koreader:master Jul 6, 2025
4 checks passed
@Commodore64user

Copy link
Copy Markdown
Member

wow, what's up with that huge purple reviewable thingy?

@benoit-pierre

Copy link
Copy Markdown
Member

Yeah, it used to be smaller… So annoying.

@Frenzie

Frenzie commented Jul 7, 2025

Copy link
Copy Markdown
Member

I imagine they'll fix it soon. It's hard to miss.

0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
…#14028)

When the contents of the HtmlBoxWidget changed the page boxes were not invalidated, this resulted the highlight to show up at a wrong coordinate.

Fixes koreader#14013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

quickdictlookup parses incorrect result

5 participants