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: close overlay drawer on click #19984

Merged
merged 2 commits into from
Sep 20, 2024
Merged

fix: close overlay drawer on click #19984

merged 2 commits into from
Sep 20, 2024

Conversation

caalador
Copy link
Contributor

Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823

Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823
This in case the flag has
for some reason not been
reset after a click.
@caalador caalador marked this pull request as ready for review September 19, 2024 07:42
Copy link

sonarcloud bot commented Sep 19, 2024

Copy link

Test Results

1 137 files  ± 0  1 137 suites  ±0   1h 9m 33s ⏱️ - 12m 0s
7 399 tests ± 0  7 349 ✅ ± 0  50 💤 ±0  0 ❌ ±0 
7 771 runs  +51  7 711 ✅ +51  60 💤 ±0  0 ❌ ±0 

Results for commit c6b1c50. ± Comparison against base commit bf420c4.

Copy link
Contributor

@tepi tepi left a comment

Choose a reason for hiding this comment

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

Tested to work fine once again

@tepi tepi merged commit 12633cd into main Sep 20, 2024
26 checks passed
@tepi tepi deleted the issues/19823-drawer-closing branch September 20, 2024 13:22
vaadin-bot pushed a commit that referenced this pull request Sep 20, 2024
* fix: close overlay drawer on click

Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823

* reset fromAnchor for navigation

This in case the flag has
for some reason not been
reset after a click.
vaadin-bot pushed a commit that referenced this pull request Sep 20, 2024
* fix: close overlay drawer on click

Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823

* reset fromAnchor for navigation

This in case the flag has
for some reason not been
reset after a click.
vaadin-bot added a commit that referenced this pull request Sep 20, 2024
* fix: close overlay drawer on click

Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823

* reset fromAnchor for navigation

This in case the flag has
for some reason not been
reset after a click.

Co-authored-by: caalador <mikael.grankvist@vaadin.com>
vaadin-bot added a commit that referenced this pull request Sep 20, 2024
* fix: close overlay drawer on click

Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823

* reset fromAnchor for navigation

This in case the flag has
for some reason not been
reset after a click.

Co-authored-by: caalador <mikael.grankvist@vaadin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vaadin 24.4.9 RouterLink in Drawer
4 participants