Skip to content
New issue

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

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

Already on GitHub? # to your account

feat(datagrid): add string filter label #1645

Merged
merged 24 commits into from
Jan 16, 2025

Conversation

valentin-mladenov
Copy link
Contributor

@valentin-mladenov valentin-mladenov commented Dec 13, 2024

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • If applicable, have a visual design approval

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Datagrid filter dialog window have no title.
image

Issue Number: CDE-2483

What is the new behavior?

Datagrid string filter dialog window have label similar to number filter dialog.
image

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@valentin-mladenov valentin-mladenov requested a review from a team December 13, 2024 09:05
@valentin-mladenov valentin-mladenov self-assigned this Dec 13, 2024
@valentin-mladenov valentin-mladenov marked this pull request as ready for review December 13, 2024 09:05
Copy link
Contributor

github-actions bot commented Dec 13, 2024

👋 @valentin-mladenov,

  • 🙏 The Clarity team thanks you for opening a pull request
  • 🎉 The build for this PR has succeeded
  • 🔍 The PR is now ready for review
  • 🍿 In the meantime, view a preview of this PR
  • 🖐 You can always follow up here. If you're a VMware employee, you can also reach us on our internal Clarity Support space

Thank you,

🤖 Clarity Release Bot

Copy link
Contributor

This PR introduces visual changes: b0c3885
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout datagrid-filter-title_cde-2483
git fetch https://github.com/vmware-clarity/ng-clarity.git b0c3885762da2cfea8ca61a279a98a673559f618
git cherry-pick b0c3885762da2cfea8ca61a279a98a673559f618
git push

Copy link
Contributor

This PR introduces visual changes: 93d8fa1
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout datagrid-filter-title_cde-2483
git fetch https://github.com/vmware-clarity/ng-clarity.git 93d8fa1478ab9ad7e4f7a925fe6a024fa31b59ef
git cherry-pick 93d8fa1478ab9ad7e4f7a925fe6a024fa31b59ef
git push

Copy link
Contributor

This PR introduces visual changes: 8dd2c8d
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout datagrid-filter-title_cde-2483
git fetch https://github.com/vmware-clarity/ng-clarity.git 8dd2c8de89bebc62cfea8d1500f877b7fb8f00ac
git cherry-pick 8dd2c8de89bebc62cfea8d1500f877b7fb8f00ac
git push

Copy link
Member

@kevinbuhmann kevinbuhmann left a comment

Choose a reason for hiding this comment

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

See comments.

Also, the filter form control labels are larger than the filter popup title. I don't think that's correct.

@valentin-mladenov
Copy link
Contributor Author

valentin-mladenov commented Dec 18, 2024

See comments.

Also, the filter form control labels are larger than the filter popup title. I don't think that's correct.

The filter title is following the signpost title design spec. I'll double check with design.

# Conflicts:
#	tests/snapshots/chromium/datagrid/column-filter--column-number-filter-opened-core-dark.png
#	tests/snapshots/chromium/datagrid/column-filter--column-number-filter-opened-core-light.png
Copy link
Contributor

This PR introduces visual changes: 2d32232
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout datagrid-filter-title_cde-2483
git fetch https://github.com/vmware-clarity/ng-clarity.git 2d322326c911955c4bc44503948117750c503950
git cherry-pick 2d322326c911955c4bc44503948117750c503950
git push

@valentin-mladenov valentin-mladenov changed the title feat(datagrid): add filter title feat(datagrid): add string filter label Jan 10, 2025
Copy link
Contributor

This PR introduces visual changes: 95a3c6a
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout datagrid-filter-title_cde-2483
git fetch https://github.com/vmware-clarity/ng-clarity.git 95a3c6a0a12bd988d77269a1cb87d50c3d1d18cd
git cherry-pick 95a3c6a0a12bd988d77269a1cb87d50c3d1d18cd
git push

@valentin-mladenov
Copy link
Contributor Author

@kevinbuhmann the acceptance criteria is changed. The implementation is totaly different.

# Conflicts:
#	projects/angular/src/data/datagrid/_datagrid.clarity.scss
#	tests/snapshots/chromium/datagrid/column-filter--column-number-filter-opened-core-dark.png
#	tests/snapshots/chromium/datagrid/column-filter--column-number-filter-opened-core-light.png
#	tests/snapshots/firefox/datagrid/column-filter--column-number-filter-opened-core-dark.png
#	tests/snapshots/firefox/datagrid/column-filter--column-number-filter-opened-core-light.png
Copy link
Contributor

This PR introduces visual changes: e3af749
If these changes are intended and correct, please cherry-pick the above commit to this PR.

git checkout datagrid-filter-title_cde-2483
git fetch https://github.com/vmware-clarity/ng-clarity.git e3af749956cacad821312117f812cc6f79e8bde8
git cherry-pick e3af749956cacad821312117f812cc6f79e8bde8
git push

@valentin-mladenov valentin-mladenov merged commit 6448c98 into main Jan 16, 2025
12 checks passed
@valentin-mladenov valentin-mladenov deleted the datagrid-filter-title_cde-2483 branch January 16, 2025 08:03
Copy link
Contributor

🎉 This PR is included in version 17.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link
Contributor

github-actions bot commented Feb 8, 2025

Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.

@github-actions github-actions bot locked and limited conversation to collaborators Feb 8, 2025
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants