👒 Update page top quo component to accommodate connected dapps anchor [19867] #19929
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes #19867
Summary
Add ability to show an action button to the
page-nav
component. For the case where an account switcher is visible (on account details page).Review notes
The button is hidden behind a feature flag. Enable
wallet-connect
feature flag to see the changes.Testing notes
This is a quo level change. There are 46 usages of this component. Unfortunately, all of them need to be tested for visual correctness. I'm sorry for that 😢
Below are all the files where the component is used. It should help you find the component in the app. Feel free to ping me on Discord if you are not able to find any listed component in the app.
Namespaces that need to be tested
Platforms
Areas that maybe impacted
page-nav
componentSteps to test
wallet/wallet-connect
feature flagBefore and after screenshots comparison
status: draft