Skip to content

Conversation

@caalador
Copy link
Contributor

When running with react-router
always wait for vaadin-navigated
event before updating the title.

In these cases the title update is
always from the server.

Fixes #21171

@github-actions
Copy link

github-actions bot commented Oct 10, 2025

Test Results

1 270 files  ±0  1 270 suites  ±0   1h 14m 40s ⏱️ - 2m 32s
8 793 tests ±0  8 726 ✅ ±0  67 💤 ±0  0 ❌ ±0 
9 246 runs  +4  9 169 ✅ +4  77 💤 ±0  0 ❌ ±0 

Results for commit 5634991. ± Comparison against base commit 4b1abc8.

♻️ This comment has been updated with latest results.

@caalador caalador marked this pull request as draft October 10, 2025 10:57
@caalador caalador force-pushed the issues/21171-history-title branch from 9a2f2b0 to 8eac7fd Compare October 10, 2025 11:35
When running with react-router
always wait for vaadin-navigated
event before updating the title.

Fixes #21171
@caalador caalador force-pushed the issues/21171-history-title branch from 8eac7fd to 630c2e9 Compare October 10, 2025 11:52
@caalador caalador marked this pull request as ready for review October 11, 2025 14:15
@mshabarov mshabarov requested a review from platosha October 13, 2025 11:34
Copy link
Contributor

@platosha platosha left a comment

Choose a reason for hiding this comment

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

Looks generally good, just some cosmetic comments.

However, would it be possible to add an IT test for this?

Add missing set navigation flag places.
@vaadin-bot vaadin-bot added +1.0.0 and removed +0.0.1 labels Oct 14, 2025
@sonarqubecloud
Copy link

@caalador caalador merged commit 1777b02 into main Oct 15, 2025
29 checks passed
@caalador caalador deleted the issues/21171-history-title branch October 15, 2025 04:58
vaadin-bot pushed a commit that referenced this pull request Oct 15, 2025
When running with react-router
always wait for vaadin-navigated
event before updating the title.

Fixes #21171
vaadin-bot pushed a commit that referenced this pull request Oct 15, 2025
When running with react-router
always wait for vaadin-navigated
event before updating the title.

Fixes #21171
vaadin-bot added a commit that referenced this pull request Oct 15, 2025
When running with react-router
always wait for vaadin-navigated
event before updating the title.

Fixes #21171

Co-authored-by: caalador <[email protected]>
vaadin-bot added a commit that referenced this pull request Oct 15, 2025
When running with react-router
always wait for vaadin-navigated
event before updating the title.

Fixes #21171

Co-authored-by: caalador <[email protected]>
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.

Incorrect Browser tab titles in Chromium based browsers

4 participants