Skip to content

Fix search result header jumps#739

Merged
bryantgillespie merged 4 commits into
mainfrom
bry/fix-search-header-jump
Jun 15, 2026
Merged

Fix search result header jumps#739
bryantgillespie merged 4 commits into
mainfrom
bry/fix-search-header-jump

Conversation

@bryantgillespie

@bryantgillespie bryantgillespie commented Jun 11, 2026

Copy link
Copy Markdown
Member

Changes

  • Fixes search result hash navigation by waiting for rendered heading targets before scrolling.
  • Aligns indexed heading anchors with GitHub-style markdown slugs, including duplicate and punctuated headings.
  • Closes Search does not take you to respective header #721.

Potential Risks

  • Search index must be rebuilt for existing indexed anchors to update.

Review Notes

  • Tests cover late-rendered #deep targets and duplicate heading anchors.

@bryantgillespie bryantgillespie requested a review from a team as a code owner June 11, 2026 21:57
@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jun 15, 2026 3:22pm

Request Review

@bryantgillespie

Copy link
Copy Markdown
Member Author

@ComfortablyCoding Can you verify this fixes the scroll to heading issue from the search?

@linear-code

linear-code Bot commented Jun 15, 2026

Copy link
Copy Markdown

EDU-523

@ComfortablyCoding

Copy link
Copy Markdown
Member

@ComfortablyCoding Can you verify this fixes the scroll to heading issue from the search?

Seems to resolved it based on some basic tests 👍 🥳

@bryantgillespie bryantgillespie merged commit 969cb82 into main Jun 15, 2026
5 checks passed
@bryantgillespie bryantgillespie deleted the bry/fix-search-header-jump branch June 15, 2026 16:07
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.

Search does not take you to respective header

2 participants