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

[MV3 Enhancement] Better than refresh UX. #1234

Closed
Tracked by #1152
whizzzkid opened this issue Jul 17, 2023 · 1 comment
Closed
Tracked by #1152

[MV3 Enhancement] Better than refresh UX. #1234

whizzzkid opened this issue Jul 17, 2023 · 1 comment
Assignees
Labels
area/MV3 Issues related to Manifest V3 version kind/enhancement A net-new feature or improvement to an existing feature

Comments

@whizzzkid
Copy link
Contributor

Current implementation involves, observing the request, checking if it can be served over ipfs, adding a redirect rule and refreshing the page.

This works, but may not be the nicest UX. an alternative may be:

  • companion prompting that the page is available from my local node and allowing me to choose whether to refresh or not.

However, this won't always work, as the original request will be on it's way to be redirected to the original url. The refresh only happens if we can determine the current URL can be resolved over the local node.

@whizzzkid whizzzkid added kind/enhancement A net-new feature or improvement to an existing feature area/MV3 Issues related to Manifest V3 version labels Jul 17, 2023
@whizzzkid whizzzkid self-assigned this Jul 17, 2023
@whizzzkid
Copy link
Contributor Author

Fixed in #1239

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/MV3 Issues related to Manifest V3 version kind/enhancement A net-new feature or improvement to an existing feature
Projects
No open projects
Status: Done
Development

No branches or pull requests

1 participant