-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
New Settings: Show extension in full size view #22383
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
c33cd4a
to
fcd5fdf
Compare
17733e5
to
4b59fc0
Compare
Great feature addition! |
LGTM ! Just need to update the snapshot for e2e test One question: is it possible to append some e2e tests for this new feature ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! +1 on @DDDDDanica's comment above regarding tests
@@ -13,6 +13,7 @@ import { | |||
setFeatureFlag, | |||
setShowFiatConversionOnTestnetsPreference, | |||
setShowTestNetworks, | |||
setShowExtensionInFullSizeView, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[super nit] placement breaks alphabetical prop ordering here and other places above
1b5d411
to
0661b11
Compare
What's New - Extension Full Size View Advanced Setting (#22369)
3ed1af1
to
759cbd0
Compare
Builds ready [759cbd0]
Page Load Metrics (1327 ± 138 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm!
Description
There are people who love to use the extension in the full size view. It's a bit time consuming to always click in the extension on three dots and then on Expand view. With this new setting the extension will open in the full size view after clicking on the extension icon (except if a 3rd party opens it, e.g. signing a Swap on Uniswap with MetaMask).
Screenshots/Recordings
Testing