feat: Change WalletButton alignment from start to center#2393
Open
marcinciarka wants to merge 3 commits intoalchemyplatform:mainfrom
Open
feat: Change WalletButton alignment from start to center#2393marcinciarka wants to merge 3 commits intoalchemyplatform:mainfrom
marcinciarka wants to merge 3 commits intoalchemyplatform:mainfrom
Conversation
WalletConnect button and the `More wallets` button are aligned to the center. It looks better at the center.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR aligns the content of the WalletButton component (used for external/injected EVM wallets) to center, matching the existing alignment of WalletConnectButton and the "More wallets" button. Previously, WalletButton used justify-start which caused a visual inconsistency when these buttons appeared in the same wallet selection list.
Changes:
- Changed the Tailwind class on the
WalletButtoncomponent fromjustify-starttojustify-centerto center-align its content.
You can also share your feedback on Copilot code review. Take the survey.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
WalletConnect button and the
More walletsbutton are aligned to the center. It looks weird when external/injected wallets are aligned to the left.classNamein theWalletButtoncomponent to usejustify-centerinstead ofjustify-start, centering the button's content.PR-Codex overview
This PR focuses on updating the alignment of buttons in two components to improve their appearance. The alignment changes from
justify-starttojustify-center, which centers the buttons within their container.Detailed summary
WalletButton.tsx, theclassNameproperty changes fromjustify-starttojustify-center.SolanaWalletButton.tsx, theclassNameproperty also changes fromjustify-starttojustify-center.