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

UDapp account dropdown refactor #5404

Merged
merged 13 commits into from
Dec 20, 2024
Merged

UDapp account dropdown refactor #5404

merged 13 commits into from
Dec 20, 2024

Conversation

joeizang
Copy link
Collaborator

fixes #5400

@joeizang joeizang added the WIP label Nov 14, 2024
@joeizang joeizang self-assigned this Nov 14, 2024
Copy link

netlify bot commented Nov 14, 2024

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 4292881
🔍 Latest deploy log https://app.netlify.com/sites/remixproject/deploys/676539c20a5a3e0008604f70
😎 Deploy Preview https://deploy-preview-5404--remixproject.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

)) : <Dropdown.Item></Dropdown.Item>}
</Dropdown.Menu>
</Dropdown>
{/* <select
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes i will

@LianaHus
Copy link
Collaborator

could you also check why dropdown-<selected item is blue in Solidity plugin/Compiler configuration

@joeizang joeizang force-pushed the account-dropdown-refactor branch 3 times, most recently from 96b7472 to 110491c Compare December 15, 2024 17:02
@joeizang joeizang force-pushed the account-dropdown-refactor branch from af6415e to af6e953 Compare December 17, 2024 18:34
@joeizang
Copy link
Collaborator Author

Details

Those dropdowns are native select elements. We could convert those also but maybe in another pr

@joeizang joeizang requested a review from LianaHus December 18, 2024 09:47
@joeizang joeizang added ready-to-review PR ready to review and removed WIP labels Dec 19, 2024
@joeizang joeizang merged commit 1104b30 into master Dec 20, 2024
34 checks passed
@joeizang joeizang deleted the account-dropdown-refactor branch December 20, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-review PR ready to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor account list dropdown to custom dropdown
2 participants