Skip to content

fix: preserve highlight visibility across URL decorations#695

Open
clehene wants to merge 1 commit intoobsidianmd:mainfrom
clehene:codex/issue-652-highlight-url-matching
Open

fix: preserve highlight visibility across URL decorations#695
clehene wants to merge 1 commit intoobsidianmd:mainfrom
clehene:codex/issue-652-highlight-url-matching

Conversation

@clehene
Copy link
Copy Markdown

@clehene clehene commented Feb 17, 2026

Summary

  • add read-path URL decoration matching so highlights are found when only query/hash differs
  • keep highlight storage writes keyed by full URL
  • add normalizeUrlForMatching helper with docs and tests

Testing

  • npm test -- string-utils.test.ts

Closes #652

@clehene clehene force-pushed the codex/issue-652-highlight-url-matching branch 2 times, most recently from 1fdf0a7 to 80b337d Compare February 25, 2026 21:43
@clehene clehene closed this Feb 25, 2026
@clehene clehene deleted the codex/issue-652-highlight-url-matching branch February 25, 2026 23:59
@clehene clehene restored the codex/issue-652-highlight-url-matching branch February 26, 2026 00:51
@clehene clehene reopened this Feb 26, 2026
@clehene clehene force-pushed the codex/issue-652-highlight-url-matching branch 2 times, most recently from e9fa9cf to c56bf98 Compare February 26, 2026 03:09
@clehene clehene force-pushed the codex/issue-652-highlight-url-matching branch from c56bf98 to 37291f1 Compare February 26, 2026 04:44
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.

BUG: webclipper should ignore URL fragment identifiers (anchors) and parameters

1 participant