Skip to content

Add search functionality on collections, Features, Templates - #325

Merged
Brigit Murtaugh (bamurtaugh) merged 19 commits into
gh-pagesfrom
bamurtaugh/ignite-copilot
Feb 22, 2024
Merged

Brigit Murtaugh (bamurtaugh) merged 19 commits into
gh-pagesfrom
bamurtaugh/ignite-copilot

Conversation

@bamurtaugh

@bamurtaugh Brigit Murtaugh (bamurtaugh) commented Nov 17, 2023 •

Copy link
Copy Markdown
Member

This pull request adds search functionality for collections, Templates, and Features.

  • Added an input field for searching tables

  • Implemented JavaScript logic to filter the table rows based on the search input

These changes improve the user experience by allowing users to easily search and find specific items in each table.

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.

Thank you so much for adding these changes, this will be so helpful for folks! 🎉
Left few comments.

Comment thread collections.html
Comment thread collections.html Outdated
Comment thread collections.html
@bamurtaugh

Copy link
Copy Markdown
Member Author

Ah sorry for any confusion! This was one of my demo scenarios during Ignite and a sample PR - I'll close this 😄

@samruddhikhandale

Samruddhi Khandale (samruddhikhandale) commented Nov 20, 2023 •

Copy link
Copy Markdown
Member

Ah! 🤦‍♀️

Brigit Murtaugh (@bamurtaugh) FYI, this is really a good PR, adding a search functionality would definitely be helpful for folks.
Let me know if I could help to get this merged in.

@bamurtaugh

Copy link
Copy Markdown
Member Author

Thanks so much for the feedback Samruddhi Khandale (@samruddhikhandale) and I agree we should reopen, especially as part of devcontainers/spec#405! Let me address your feedback and then re-request review.

@bamurtaugh Brigit Murtaugh (bamurtaugh) changed the title Add search functionality and style to collection table Add search functionality on collections, Features, Templates Feb 17, 2024

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.

Looks great, thank you! ⚡

Comment thread collections.html
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