fix: make Home tool cards responsive - #178
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7126474f60
ℹ️ 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".
| @media (max-width: 1180px) { | ||
| [data-testid="stHorizontalBlock"]:has([class*="st-key-tool_card_"]) { | ||
| flex-wrap: wrap; |
There was a problem hiding this comment.
Base wrapping on the grid's available width
When the sidebar is expanded on laptop/desktop viewports just above 1180px, the main content grid is substantially narrower than the viewport, but this media query does not activate, so render_tool_section() still compresses all five columns into the same unreadable row this change intends to prevent. Use a container query for the horizontal block or raise/adjust the breakpoint to account for the sidebar rather than keying solely off viewport width.
Useful? React with 👍 / 👎.
Summary
Responsive behavior
Verification
pytest -qpassed.ruff check .passed.compileall,pip check, andgit diff --checkpassed.