Skip to content

0xlukem/footer redesign#170

Merged
eshaben merged 8 commits into
mainfrom
0xlukem/footer-redesign
Apr 15, 2026
Merged

0xlukem/footer redesign#170
eshaben merged 8 commits into
mainfrom
0xlukem/footer-redesign

Conversation

@0xlukem

@0xlukem 0xlukem commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

This pull request introduces several UI and navigation improvements to the documentation site, focusing on enhanced header functionality, updated footer layout for mobile devices, and adjustments to redirect rules for code review reference pages.

UI/UX Improvements:

  • Added a "Video Library" button to the header, linking directly to the video library section for easier access.
  • Ensured compact utility controls in the header remain visible when the desktop search bar expands, improving usability.

Footer and Layout Adjustments:

  • Updated mobile footer breakpoints and layout: adjusted media query ranges, reduced bottom padding, changed grid column structure, and improved alignment for footer elements on screens up to 1024px wide.
  • Changed the smallest mobile breakpoint from 767px to 425px for footer stacking, ensuring better layout on very small screens.

Navigation and Redirect Updates:

  • Updated several redirect rules in redirects.json so that legacy links for code review reference tools and schemas now point to the consolidated /code-reviews/reference/ page, simplifying navigation and reducing outdated links. [1] [2]

This PR needs to be merged with this one: kluster-ai/docs#276

Copilot AI review requested due to automatic review settings April 10, 2026 23:56
@0xlukem 0xlukem mentioned this pull request Apr 10, 2026
3 tasks
@0xlukem 0xlukem requested a review from eshaben April 10, 2026 23:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the documentation site’s navigation UX by adding a direct “Video Library” entry point in the header, refining responsive footer behavior for smaller screens, and simplifying legacy redirect targets for code review reference pages.

Changes:

  • Added an icon-only “Video Library” header button linking to /code-reviews/video-library/.
  • Adjusted header/search CSS so compact header utility controls remain visible when the desktop search UI expands.
  • Updated footer mobile breakpoints/layout and consolidated several legacy redirects to /code-reviews/reference/.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
redirects.json Redirect legacy tools/schema URLs to the consolidated code reviews reference page.
material-overrides/partials/header.html Add “Video Library” header button/link.
material-overrides/assets/stylesheets/kluster.css Keep header utilities visible during expanded search; refine footer responsive layout/breakpoints.
material-overrides/.icons/custom/history.svg Add custom icon asset.
material-overrides/.icons/custom/file.svg Add custom icon asset.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread material-overrides/assets/stylesheets/kluster.css
Comment thread material-overrides/assets/stylesheets/kluster.css Outdated

@eshaben eshaben left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

one last little thing

Comment thread material-overrides/assets/stylesheets/kluster.css Outdated
Comment thread material-overrides/partials/header.html Outdated
0xlukem and others added 2 commits April 15, 2026 10:24
Co-authored-by: Erin Shaben <eshaben@icloud.com>
Co-authored-by: Erin Shaben <eshaben@icloud.com>
@0xlukem 0xlukem requested a review from eshaben April 15, 2026 13:43
@eshaben eshaben merged commit 7831a50 into main Apr 15, 2026
@eshaben eshaben deleted the 0xlukem/footer-redesign branch April 15, 2026 20:28
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.

3 participants