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

Fix Filters In Horizontal Filters #1512

Merged
merged 4 commits into from
Sep 3, 2024
Merged

Fix Filters In Horizontal Filters #1512

merged 4 commits into from
Sep 3, 2024

Conversation

cwolf10
Copy link
Collaborator

@cwolf10 cwolf10 commented Aug 30, 2024

Description

Change structure of horizontal filters to allow formly forms being used for filters to appear in new popover. Also made a few changes to the popover configuration to address bugs found during testing.

Motivation and Context

During demo it was initially noted that certain filters in the horizontal filters were not displaying.

Type of Change (Select One and Apply Github Label)

  • Bug fix (non-breaking change which fixes an issue) -> Apply bugfix label
  • New feature (non-breaking change which adds functionality) -> Apply enhancement label
  • Breaking change (fix or feature that would cause existing functionality to change) -> Apply breaking label

Screenshots (if appropriate):

Which browsers have you tested?

  • Internet Explorer 11
  • Edge
  • Chrome
  • Firefox
  • Safari

Checklist:

Copy link

github-actions bot commented Aug 30, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-09-03 20:56 UTC

</formly-form>
<div *ngFor="let field of fields">
<div
*ngIf="!field.hide"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why did we remove this !dialogRef condition, this is for the mobile behaviour, can you check the mobile view, it's not broken

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just checked the horizontal filters, mobile view still works fine after this change.

@cwolf10 cwolf10 merged commit b528251 into master Sep 3, 2024
7 checks passed
@cwolf10 cwolf10 deleted the popover-horizontal branch September 5, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants