Skip to content

Fix: Improve visibility of Related Videos carousel navigation (#1163)#1164

Open
piyushtripathi9424 wants to merge 1 commit into
layer5io:masterfrom
piyushtripathi9424:fix/docs-carousel-navigation-visibility-1163
Open

Fix: Improve visibility of Related Videos carousel navigation (#1163)#1164
piyushtripathi9424 wants to merge 1 commit into
layer5io:masterfrom
piyushtripathi9424:fix/docs-carousel-navigation-visibility-1163

Conversation

@piyushtripathi9424

@piyushtripathi9424 piyushtripathi9424 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Description

This PR fixes Issue #1163 by making the "Related Videos" carousel navigation more visible and accessible.

Key Changes

  • Arrows: Applied filter: invert(100%) contrast(200%) and enlarged the arrows to ensure high contrast without needing bulky circular backgrounds.
  • Pagination: Highlighted the active slide indicator with the primary teal color and increased its width.
  • Performance & Scoping: Optimized CSS transitions per reviewer feedback and safely scoped all new styles to .related-videos.

Screenshot

Screenshot 2026-07-14 at 4 17 45 AM

Fixes #1163

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request refactors and enhances the carousel styles within the .related-videos class in _videos_project.scss. The review feedback recommends replacing the generic transition: all properties with explicit transitions for specific properties (such as background-color, box-shadow, transform, and width) to improve browser rendering performance.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread assets/scss/_videos_project.scss Outdated
Comment thread assets/scss/_videos_project.scss Outdated
@piyushtripathi9424 piyushtripathi9424 force-pushed the fix/docs-carousel-navigation-visibility-1163 branch from bf4fb41 to c6507b8 Compare July 13, 2026 17:41
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Preview deployment: https://docs.layer5.io/pr-preview/pr-1164/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

@piyushtripathi9424 piyushtripathi9424 force-pushed the fix/docs-carousel-navigation-visibility-1163 branch from da47b3f to e19fe2f Compare July 13, 2026 18:03

@yi-nuo426 yi-nuo426 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.

Thanks. Don't encircle the arrows. Just invert their coloring to be of high contrast.

@piyushtripathi9424 piyushtripathi9424 force-pushed the fix/docs-carousel-navigation-visibility-1163 branch from c1674d1 to 3b9ef8f Compare July 13, 2026 22:29
Fixes layer5io#1163 by increasing the contrast, size, and hover effects of the carousel navigation arrows and pagination indicators to ensure visibility across both light and dark modes.

Signed-off-by: piyushtripathi9424 <piyush.miraibanglore@gmail.com>
@piyushtripathi9424 piyushtripathi9424 force-pushed the fix/docs-carousel-navigation-visibility-1163 branch from 3b9ef8f to 4780c1f Compare July 13, 2026 22:47
@piyushtripathi9424

Copy link
Copy Markdown
Contributor Author

Thanks. Don't encircle the arrows. Just invert their coloring to be of high contrast.

Inverted as requested.

@Sbragul26 Sbragul26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could we make the icon a bit smaller? It is bigger than the one on the production site and grows even more on hover. I think just changing the color for better contrast would be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] Improve visibility of carousel navigation

3 participants