Skip to content

Improve responsive styling for integration category chips#7727

Open
shteypandey28-hue wants to merge 1 commit into
layer5io:masterfrom
shteypandey28-hue:fix/integration-category-chip-styling
Open

Improve responsive styling for integration category chips#7727
shteypandey28-hue wants to merge 1 commit into
layer5io:masterfrom
shteypandey28-hue:fix/integration-category-chip-styling

Conversation

@shteypandey28-hue
Copy link
Copy Markdown
Contributor

@shteypandey28-hue shteypandey28-hue commented May 13, 2026

Summary

This PR refactors the category chip UI for better responsiveness, accessibility, and mobile usability.

Changes

  • Inverted active/inactive chip styling
  • Replaced react-slick with responsive flexbox + horizontal scroll
  • Added semantic buttons with ARIA attributes and data-category
  • Improved keyboard focus and reduced-motion accessibility
  • Added mobile scroll snap and overflow fade indicator
  • Enhanced hover/active transitions with Keppel theme colors
  • Fixed mobile sizing, clipping, and overflow issues
  • Added full dark mode support using existing theme tokens

Description

Fixes #

Notes for Reviewers

  • Removed react-slick dependency
  • Tested across mobile and desktop layouts
  • Signed commits included

@shteypandey28-hue shteypandey28-hue force-pushed the fix/integration-category-chip-styling branch from 780e1ac to 8a3bcf7 Compare May 13, 2026 16:16
- Invert chip styling: active chips now use light bg with border,
  inactive chips use dark filled bg (fixes layer5io#7593)
- Replace react-slick slider with responsive flexbox/scroll layout
- Add proper button elements with data-category attrs for robust filtering
- Add ARIA attributes (role, aria-pressed, aria-label) for accessibility
- Add focus-visible ring, reduced-motion support
- Add mobile horizontal scroll with snap and overflow fade indicator
- Add smooth hover/active transitions with Keppel theme colors
- Touch-friendly sizing on mobile with no clipping or overflow issues
- Full dark mode support using existing theme tokens

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>
@shteypandey28-hue shteypandey28-hue force-pushed the fix/integration-category-chip-styling branch from 8a3bcf7 to b62d11c Compare May 13, 2026 16:17
@github-actions
Copy link
Copy Markdown
Contributor

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

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

@shteypandey28-hue
Copy link
Copy Markdown
Contributor Author

shteypandey28-hue commented May 13, 2026

@leecalcote @MUzairS15 @CodexRaunak @YASHMAHAKAL This PR is ready for review.

Fixes #7593 — Inverts category chip styling (active → light, inactive → dark), replaces the react-slick slider with a responsive flexbox layout, and adds accessibility improvements (keyboard navigation, ARIA attributes, focus-visible ring).

All CI checks are passing ✅

@shteypandey28-hue
Copy link
Copy Markdown
Contributor Author

Here's the full visual summary in the artifact above. The key screenshots:

Light Mode — The "ALL" chip is shown with the new selected styling: light mint-green background + Keppel green border (inverted from the old dark bg):

WhatsApp Image 2026-05-13 at 22 23 47

Dark Mode — Same chip with dark theme support: teal-tinted background + Keppel green border:

WhatsApp Image 2026-05-13 at 22 23 59

The lite build excludes integrations data so only "ALL (0)" is visible. On the full production build (or the PR preview deployment), all 10+ category chips will render with the same pill-shaped, properly wrapped layout.

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.

1 participant