Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

fix(filter-field): Fixes an issue where two filterfield flaps were open at the same time. #849

Merged
merged 1 commit into from
Apr 14, 2020

Conversation

tomheller
Copy link
Collaborator

@tomheller tomheller commented Apr 8, 2020

Pull Request

To ensure that only one filterfield flap is open at the same time, we need to store
the instance of the open flap in a global / module scope. If another filter field is focussed,
it should close the panels of the currently opened one and then reassign itself into the
open position.

Fixes #840

Type of PR

Bugfix

Checklist

  • I have read the CONTRIBUTING doc and I follow the PR guidelines
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@tomheller tomheller self-assigned this Apr 8, 2020
thomaspink
thomaspink previously approved these changes Apr 8, 2020
@github-actions
Copy link

github-actions bot commented Apr 8, 2020

Deploy preview for website ready!

Built with commit f718c1c

https://browser-hkq45twn4.now.sh

@tomheller
Copy link
Collaborator Author

I have added the _filterLabelsClose() call also to the ngOnDestroy, to make sure all labels are being closed when the component is being destroyed and the currentlyOpenFilterfield also resets.

ffriedl89
ffriedl89 previously approved these changes Apr 9, 2020
libs/examples/src/index.ts Outdated Show resolved Hide resolved
libs/barista-components/filter-field/src/filter-field.ts Outdated Show resolved Hide resolved
lukasholzer
lukasholzer previously approved these changes Apr 9, 2020
@tomheller tomheller dismissed stale reviews from lukasholzer and ffriedl89 via fdc3ff4 April 9, 2020 08:54
@tomheller tomheller force-pushed the fix/filter-field-not-closes-on-click-outside branch from 032d9de to fdc3ff4 Compare April 9, 2020 08:54
thomaspink
thomaspink previously approved these changes Apr 9, 2020
…en at the same time.

To ensure that only one filterfield flap is open at the same time, we need to store
the instance of the open flap in a global / module scope. If another filter field is focussed,
it should close the panels of the currently opened one and then reassign itself into the
open position.

Fixes #840
@sonarcloud
Copy link

sonarcloud bot commented Apr 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@ffriedl89 ffriedl89 added the pr: merge-ready This PR is ready to be merged label Apr 10, 2020
@github-actions github-actions bot added target: minor This PR is targeted for the next minor release target: patch This PR is targeted for the next patch release labels Apr 10, 2020
@lukasholzer lukasholzer merged commit b5c470e into master Apr 14, 2020
@lukasholzer lukasholzer deleted the fix/filter-field-not-closes-on-click-outside branch April 14, 2020 05:29
@github-actions
Copy link

The auto cherry picker succeeded.Cherry picked to minor branch 6.x. New head is now 85feb21
Cherry picked to patch branch 6.2.x. New head is now 504d054

tomheller added a commit that referenced this pull request Nov 19, 2020
When clicking another combobox dropdown arrow (which stopped the event propagation), the
autocomplete outside click was never triggered, preventing the blurred combobox from closing.

A similar solution as in the filter-field #849 was applied to this.

Fixes APM-269028
lukasholzer pushed a commit that referenced this pull request Nov 20, 2020
When clicking another combobox dropdown arrow (which stopped the event propagation), the
autocomplete outside click was never triggered, preventing the blurred combobox from closing.

A similar solution as in the filter-field #849 was applied to this.

Fixes APM-269028
github-actions bot pushed a commit that referenced this pull request Nov 20, 2020
When clicking another combobox dropdown arrow (which stopped the event propagation), the
autocomplete outside click was never triggered, preventing the blurred combobox from closing.

A similar solution as in the filter-field #849 was applied to this.

Fixes APM-269028
github-actions bot pushed a commit that referenced this pull request Nov 20, 2020
When clicking another combobox dropdown arrow (which stopped the event propagation), the
autocomplete outside click was never triggered, preventing the blurred combobox from closing.

A similar solution as in the filter-field #849 was applied to this.

Fixes APM-269028
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge-ready This PR is ready to be merged target: minor This PR is targeted for the next minor release target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filter field: autocomplete stays open when clicking into another filter field
4 participants