MetaMask / metamask-extension

:globe_with_meridians: :electric_plug: The MetaMask browser extension enables browsing Ethereum blockchain enabled websites
https://metamask.io
Other
12.04k stars 4.91k forks source link

Replace deprecated Dialog component with BannerAlert from the component-library #20463

Open georgewrmarshall opened 1 year ago

georgewrmarshall commented 1 year ago

Description

Currently, the extension is using an outdated Dialog component, which needs to be replaced with the new BannerAlert component.

This is a massive undertaking by itself and creating a single PR would be too large. Smaller PRs can be submitted against this issue to ensure easier review and gradual improvements.

Technical Details

Acceptance Criteria

If the acceptance criteria is not met, PRs may be closed.

Difficulty: Intermediate

Good first issue for: External contributors who are familiar with running the extension locally, have knowledge of React, component props, Jest tests, linting, and Storybook, and want to contribute to improving the cohesiveness of UI in the extension

KeerthanaR29 commented 1 year ago

Please assign this issue to me!I would like to work on it!

georgewrmarshall commented 1 year ago

Hey @KeerthanaR29, this issue represents a large effort of work from multiple contributors. Feel free to open a PR that references this issue.

rage73 commented 6 months ago

Hi, I would like to work on this seems like its not assigned to anyone currently.