Skip to content

Conversation

@Sekar-C-Mca
Copy link
Contributor

Description

This PR implements the enhancement requested in issue #6935. It adds support for filtering FAQs by both category and subcategory, with dynamic subcategory options that update based on selected categories.

Changes

  • Added subcategory filtering support to the FAQ component
  • Implemented dynamic subcategory options based on selected categories
  • Used SistentThemeProvider and theme tokens for consistent styling
  • Improved filter display and user experience
image image image image

This commit implements the enhancement requested in issue layer5io#6935. It adds support for filtering FAQs by both category and subcategory, with dynamic subcategory options based on selected categories. Uses SistentThemeProvider and theme tokens for consistent styling.

Signed-off-by: Sekar <sekarmca2024.com>
Signed-off-by: Sekar C Mca <sekarmca2024@gmail.com>
@Sekar-C-Mca Sekar-C-Mca force-pushed the feature/enhance-faq-subcategory-filtering branch from c24f737 to 35a9480 Compare October 22, 2025 13:09
@l5io
Copy link
Contributor

l5io commented Oct 22, 2025

🚀 Preview for commit 35a9480 at: https://68f8dab8e7a9a4451541103b--layer5.netlify.app

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

Everything looks good, but it always shows all items ( refer image ), please fix that. Also, in the current implementation, both category and subcategory need to be selected to display results. Instead, make it so that selecting a category shows all its subcategories by default, and selecting a subcategory should remain optional.

image

…support

- Use Box component from @sistent/sistent with theme tokens
- Replace hard-coded colors with background.default and divider tokens
- Implement Collapse component properly for filter transitions
- Remove unused Row and Col imports
- Fix dark mode visibility issues with proper theme-aware styling

Signed-off-by: Sekar C Mca <sekarmca2024@gmail.com>
@l5io
Copy link
Contributor

l5io commented Oct 26, 2025

🚀 Preview for commit c50cc9a at: https://68fe5e3522142b0cea778fdd--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT
Add it as an agenda item to the meeting minutes, if you would 😄

@l5io
Copy link
Contributor

l5io commented Oct 27, 2025

🚀 Preview for commit 14c22a1 at: https://68ff92d81d84bf78b016bca6--layer5.netlify.app

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit ed8c6cb into layer5io:master Oct 27, 2025
4 checks passed
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.

4 participants