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

Browser: fully eliminate reloading issues #5576

Merged
merged 8 commits into from
Mar 29, 2024

Conversation

christianbaroni
Copy link
Member

What changed (plus any additional context for devs)

  • Prevents WebView reloading when switching tabs
  • Improves screenshot display logic
  • Mostly fixes canGoBack logic and adds a back button to the context menu for now
  • Adds apple-touch-icon support (will likely want to update how this is managed with the state refactor)
  • Fixes loading bar bugs by moving the loading bar outside of the webview component — no longer creating a new progress bar for each tab
  • Misc. UI cleanup

Screen recordings / screenshots

RPReplay_Final1711622304.mp4

What to test

(This prevents the URL in the SearchInput from updating correctly in certain cases - will need to address another way)
Can now pass a shared value string as the sole child of AnimatedText
Copy link
Member

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

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

💯thanks for the comments

src/components/DappBrowser/BrowserTab.tsx Outdated Show resolved Hide resolved
@brunobar79 brunobar79 merged commit 8518cb6 into develop Mar 29, 2024
5 of 6 checks passed
@brunobar79 brunobar79 deleted the @christian/browser-nav-state-handling branch March 29, 2024 22:27
BrodyHughes added a commit that referenced this pull request Apr 2, 2024
* 'develop' of github.com:rainbow-me/rainbow:
  Only hold the active tab ref in BrowserContext (#5579)
  Dapp browser: disable tab closing for empty state (#5573)
  Browser: fix ref assignment, back/forward navigation (#5578)
  Browser: fully eliminate reloading issues (#5576)
  browser: static trending dapps (#5561)
  bump swaps sdk (#5574)
  fix gitignore (#5571)
  bump (#5570)
  Fix browser context menu not updating (#5569)
  ⚡️ Fast browser (#5566)
  [APP-1049]: (feat): Backups V2 (#5310)
  fix: search by contract address (#5563)
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.

None yet

3 participants