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

Vaadin 24.4.9 RouterLink in Drawer #19823

Closed
alexanoid opened this issue Aug 24, 2024 · 4 comments · Fixed by #19984
Closed

Vaadin 24.4.9 RouterLink in Drawer #19823

alexanoid opened this issue Aug 24, 2024 · 4 comments · Fixed by #19984

Comments

@alexanoid
Copy link

alexanoid commented Aug 24, 2024

Description of the bug

Previous similar bug #18736

With Vaadin 24.4.9, when I click on the RouterLink which points to the page where I currently am, the Drawer on the mobile view does not close, unlike in version 24.4.0.beta5 :

drawer

Expected behavior

The drawer on the mobile view should be closed after the click, even if it is pointed to the page where the user currently is

Minimal reproducible example

n/a

Versions

  • Vaadin / Flow version: 24.4.9
  • Java version: 17
@mshabarov
Copy link
Contributor

This is probably a regression after some other fix, as #18736 aimed to fix the same problem.

@tepi
Copy link
Contributor

tepi commented Sep 13, 2024

Confirmed this is a regression on latest versions and on 24.5 branch as well. Investigating.

caalador added a commit that referenced this issue Sep 19, 2024
Close the overlay drawer
even when clicking on
same link and no actual
navigation happens.

Fixes #19823
@caalador caalador self-assigned this Sep 19, 2024
@tepi tepi closed this as completed in 12633cd Sep 20, 2024
vaadin-bot pushed a commit that referenced this issue 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 issue 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 issue 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 <[email protected]>
vaadin-bot added a commit that referenced this issue 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 <[email protected]>
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.5.0.beta2 and is also targeting the upcoming stable 24.5.0 version.

@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.4.13.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment