Skip to content

Fix blog route matching article header#769

Open
Sheraff wants to merge 8 commits intoTanStack:mainfrom
Sheraff:fix-blog-route-matching-article-header
Open

Fix blog route matching article header#769
Sheraff wants to merge 8 commits intoTanStack:mainfrom
Sheraff:fix-blog-route-matching-article-header

Conversation

@Sheraff
Copy link
Contributor

@Sheraff Sheraff commented Mar 17, 2026

Just adding a header image because with the new layout, the previous one just wasn't nice anymore (wasn't the best before either)

before
Screenshot 2026-03-17 at 20 34 34

after
Screenshot 2026-03-17 at 20 34 44

Summary by CodeRabbit

  • Documentation
    • Updated a blog post’s header image (new header artwork and alt text).
    • Added a new illustrative image after the opening paragraph with descriptive alt text to highlight a performance figure and improve visual presentation.

@netlify
Copy link

netlify bot commented Mar 17, 2026

👷 Deploy request for tanstack pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 3494f75

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a54777e1-9f7e-4c59-a001-737e8a479f3e

📥 Commits

Reviewing files that changed from the base of the PR and between 6964b7c and 3494f75.

⛔ Files ignored due to path filters (1)
  • public/blog-assets/tanstack-router-route-matching-tree-rewrite/header.jpg is excluded by !**/*.jpg
📒 Files selected for processing (1)
  • src/blog/tanstack-router-route-matching-tree-rewrite.md
✅ Files skipped from review due to trivial changes (1)
  • src/blog/tanstack-router-route-matching-tree-rewrite.md

📝 Walkthrough

Walkthrough

Replaced the article header image: header.png (alt "Big performance number") → header.jpg (alt "Fantasy landscape illustration used as the article header"). Inserted a new big-number.png image (alt "Big performance number") after the opening paragraph with an extra blank line before the next section header.

Changes

Cohort / File(s) Summary
Blog post image updates
src/blog/tanstack-router-route-matching-tree-rewrite.md
Replaced header image file and alt text (header.pngheader.jpg, alt updated). Added a new embedded image line for big-number.png with alt "Big performance number" after the opening paragraph and added an extra blank line before the following section header.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through markdown dew,
Swapped a header view for something new,
Dropped a big-number image in the lead,
Whitespace tidy — a tiny deed,
Hooray for pixels, crisp and true! 🎨✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix blog route matching article header' accurately describes the main change: replacing and updating header images in the blog article to improve visual appearance with the new layout.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

@Sheraff Sheraff marked this pull request as ready for review March 17, 2026 19:35
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6964b7c572

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/blog/tanstack-router-route-matching-tree-rewrite.md`:
- Line 9: The image markdown currently uses a generic alt text "Header" in the
line containing
"![Header](/blog-assets/tanstack-router-route-matching-tree-rewrite/header.png)";
replace that alt text with a meaningful description (e.g., "Fantasy landscape
illustration used as the article header") or make it decorative by using an
empty alt (`![](/blog-assets/.../header.png)`), updating the markdown line
accordingly so screen readers receive either the descriptive text or treat the
image as decorative.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d5adcc53-e648-4fae-a1ed-ecf8e6cfea9c

📥 Commits

Reviewing files that changed from the base of the PR and between fb4dc25 and 6964b7c.

⛔ Files ignored due to path filters (2)
  • public/blog-assets/tanstack-router-route-matching-tree-rewrite/big-number.png is excluded by !**/*.png
  • public/blog-assets/tanstack-router-route-matching-tree-rewrite/header.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • src/blog/tanstack-router-route-matching-tree-rewrite.md

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.

2 participants