- Notifications
You must be signed in to change notification settings - Fork928
Closed
Description
What is your suggestion?
CurrentlyResetPasswordDialog
has its own component, whereas we have a better more generalizedConfirmDialog
component. The suggestion is to use theConfirmDialog
component to create the reset password dialog box.
Why do you want this feature?
Better code structure, removing anti-patterns of keeping two different components doing the same thing. This will also keep the same styles for both types of dialogs.
Currently, they look slightly different:
Are there any workarounds to get this functionality today?
Yes, the component exists and works fine.
Are you interested in submitting a PR for this?
Yes