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

654 gs lineage filter add button to delete current input #662

Merged

Conversation

JonasKellerer
Copy link
Collaborator

resolves #654

Summary

Downshift is now used for the lineage filter.

Breaking change: renames initialValue to value to be in line with the generic input element.

Screenshot

The input with delete button
grafik

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by an appropriate test.

Copy link

vercel bot commented Jan 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dashboard-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 6:59am

Copy link
Contributor

github-actions bot commented Jan 18, 2025

This is a preview of the changelog of the next release. If this branch is not up-to-date with the current main branch, the changelog may not be accurate. Rebase your branch on the main branch to get the most accurate changelog.

Note that this might contain changes that are on main, but not yet released.

Changelog:

0.13.0 (2025-01-21)

⚠ BREAKING CHANGES

  • components: Rename initialValue to value for gs-lineage-filter
  • components: Rename initialValue to value for text input

Features

  • components: gs-lineage-filter: rename initialValue to value (b22b710)
  • components: gs-lineage-filter: use downshift for selector (e8f1657), closes #654

Code Refactoring

  • components: extract downshift combobox from location filter and text input (dfe105e), closes #660

@JonasKellerer JonasKellerer linked an issue Jan 20, 2025 that may be closed by this pull request
@JonasKellerer JonasKellerer force-pushed the 660-extract-dropdown-from-text-input-and-location-filter branch from 3aefe97 to 70cb363 Compare January 20, 2025 09:16
Base automatically changed from 660-extract-dropdown-from-text-input-and-location-filter to main January 20, 2025 09:29
@JonasKellerer JonasKellerer force-pushed the 654-gs-lineage-filter-add-button-to-delete-current-input branch from 00a9074 to f3a57f5 Compare January 20, 2025 09:44
Copy link
Collaborator

@anna-parker anna-parker left a comment

Choose a reason for hiding this comment

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

looks good to me and when I tested the preview it also looks ok - just one naming question/ suggestion

components/src/preact/locationFilter/location-filter.tsx Outdated Show resolved Hide resolved
@fengelniederhammer
Copy link
Collaborator

Just a little commit message nitpick: I usually prefer to have the exact component tag in the message. "lineage filter" seems a little too unspecific to me. That's why I usually write messages that target a single component in the style gs-lineage-filter: change something important.

@JonasKellerer JonasKellerer force-pushed the 654-gs-lineage-filter-add-button-to-delete-current-input branch from 95bc4e6 to 469f475 Compare January 21, 2025 06:58
Jonas Zarzalis (TNG) added 2 commits January 21, 2025 07:58
@JonasKellerer JonasKellerer force-pushed the 654-gs-lineage-filter-add-button-to-delete-current-input branch from 469f475 to b22b710 Compare January 21, 2025 06:58
@JonasKellerer JonasKellerer merged commit 1ff38e4 into main Jan 21, 2025
7 checks passed
@JonasKellerer JonasKellerer deleted the 654-gs-lineage-filter-add-button-to-delete-current-input branch January 21, 2025 07:20
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.

gs-lineage-filter: add button to delete current input
3 participants