Page MenuHomePhabricator

Try adding inline errors instead of alerts for missing amount/opt-in
Closed, ResolvedPublic


As suggested by @Krinkle in T205683. This would be less jarring and more consistent with payments forms. It's also more important now that there can be multiple required choices on the form (amount + opt-in).

We should probably test this on banners too.

Event Timeline

Pcoombe created this task.Sep 28 2018, 11:28 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 28 2018, 11:28 AM

I've done this for the email opt-in now. Amount will be a bit more tricky as there's multiple possible error messages, and some need to include min/max amounts.

We now have this for amount, payment method and email opt-in across all our banners: example

The last place to add it is on for amounts. This is tricky and we're entering peak banner season, so it will probably have to wait until next quarter.

Pcoombe closed this task as Resolved.Mar 28 2019, 8:25 PM
Pcoombe claimed this task.

This is done on now as well. See the donationForm.validate method in