Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(platform): add dropdown menu for predefined filter options #12859

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mohammad-nmeri
Copy link

Description

This PR enhances the filtering dialog in the Platform App by adding a dropdown for selecting predefined filter options. It introduces a new filterValuesOptions attribute in the columns component, which expects a string array of predefined filter values to be sent when defining a column. These values will be displayed in the filter dropdown, improving the filtering experience. Additionally, a new "Array" data type for columns is implemented to support this functionality.

Screenshots

main

Before:

before

After:

after

Copy link

netlify bot commented Dec 19, 2024

Check the deploy log for errors here: https://app.netlify.com/sites/fundamental-ngx/deploys

Name Link
🔨 Latest commit 9070c26
🔍 Latest deploy log https://app.netlify.com/sites/fundamental-ngx/deploys/67642147bc6f7c000841a0cb

Copy link
Member

@mikerodonnell89 mikerodonnell89 left a comment

Choose a reason for hiding this comment

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

@mohammad-nmeri Sorry, but before we can review, commit messages must adhere to these rules for release log generation purposes: https://github.com/SAP/fundamental-ngx/blob/main/CONTRIBUTING.md#commit

@mohammad-nmeri mohammad-nmeri changed the title feat(filtering): add dropdown menu for predefined filter options feat(platform): add dropdown menu for predefined filter options Jan 7, 2025
@mohammad-nmeri
Copy link
Author

@mikerodonnell89, apologies for the oversight. I have updated the commit message

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.

2 participants