Skip to content

Fix errors in loading of help view#1501

Merged
esimkowitz merged 5 commits into
mainfrom
evan/fix-helpview-fail-load
Dec 12, 2024
Merged

Fix errors in loading of help view#1501
esimkowitz merged 5 commits into
mainfrom
evan/fix-helpview-fail-load

Conversation

@esimkowitz
Copy link
Copy Markdown
Contributor

When clicking on one of the cards from the home page of the docsite and when initially loading the home page of the docsite, the webviewTag was emitting a did-frame-navigate event, which we weren't tracking. This meant that we were not properly recording that a navigation had occurred. This caused three separate issues:

  • We were never setting the meta url for the block, which meant that when you navigated to a different tab or reloaded the app, you'd lose what page you were last on.
  • The site would reload after we fixed a broken docsite url, but we wouldn't remove the error text that blocks the broken site.
  • Clicking on the "open in external browser" button wouldn't do anything.

@esimkowitz esimkowitz merged commit 9ce0093 into main Dec 12, 2024
@esimkowitz esimkowitz deleted the evan/fix-helpview-fail-load branch December 12, 2024 00:09
xxyy2024 pushed a commit to xxyy2024/waveterm_aipy that referenced this pull request Jun 24, 2025
When clicking on one of the cards from the home page of the docsite and
when initially loading the home page of the docsite, the webviewTag was
emitting a `did-frame-navigate` event, which we weren't tracking. This
meant that we were not properly recording that a navigation had
occurred. This caused three separate issues:

- We were never setting the meta url for the block, which meant that
when you navigated to a different tab or reloaded the app, you'd lose
what page you were last on.
- The site would reload after we fixed a broken docsite url, but we
wouldn't remove the error text that blocks the broken site.
- Clicking on the "open in external browser" button wouldn't do
anything.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant