Repository navigation
refactor: Docs layout - #859
Conversation
github-actions
Bot
added
the
🧱 Refactor
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
React Doctor found no new issues. 🎉 Reviewed by React Doctor for commit |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2a932ee7f5
ℹ️ 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".
| return ( | ||
| <nav | ||
| aria-label="Previous and next page" | ||
| className="mt-10 flex flex-col gap-4" |
There was a problem hiding this comment.
The new footer mixes mt-10 with gap-4, and repeats the same pattern with gap-6 and padding on the next-page link. Split the outer spacing onto a wrapper or use one spacing mechanism so this component follows the repository's explicit layout convention.
AGENTS.md reference: AGENTS.md:L73-L73
Useful? React with 👍 / 👎.
| const item = itemOf(event.target); | ||
| if (item && event.pointerType === "mouse") showHover(item); |
There was a problem hiding this comment.
When the mouse moves from a menu button onto padding, a menu gap, or empty scroll space inside SidebarContent, this handler receives a non-item target but does nothing; pointerleave does not fire until the pointer exits the entire content area. The hover highlight therefore remains stuck on the previous item even though it is no longer hovered, so mouse events without an item should clear the hover state.
Useful? React with 👍 / 👎.
1 participant
Improving Documentation
pnpm lint:fixto fix formatting issues before opening the PR.Description
What?
Why?