-
Notifications
You must be signed in to change notification settings - Fork 41
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
Refactor report abuse form to use DismissibleTextForm #10834
Comments
Bumping priority because other components use auto-save, so it'd be nice to have auto-save for that too. |
I am trying to refactor |
Yes, they were designed at different times. I would suggest making it look consistent with 'report user abuse' because that one uses newer button styles. |
@kumar303 @willdurand AMO stage - new design AMO prod - old design which is going to be replaced |
Part of the report abuse form was extracted into a reusable component. It landed in #10833
The
ReportAbuseButton
component should now useDismissibleTextForm
. Take a look atReportUserAbuse
for an example of how to do the conversion.For QA:
The text was updated successfully, but these errors were encountered: