Improvements to custom ASLAY reason

[Tiagoquix]

In the ASLAY pop-up that has many pre-written reasons that only need to be checked to be taken into effect, there's also the custom reason field with a checkbox that allows an admin to write a custom reason for the ASLAY.

It is very useful and it could even be pre-checked by default, so admins don't forget to check it when they want to write a custom reason.

However, if an admin marks the checkbox and doesn't type anything, the ASLAY reason will be a bit ugly: it will be something such as + Random DeathMatch.

Therefore, I have some suggestions to improve this experience:

1. Remove the checkbox to write custom reasons and replace it with a text such as Custom reason: (therefore, it will always be available and will not require to mark the checkbox first because there will be no checkbox).
2. Detect if there's anything typed in the custom reason field. If there's nothing, then the reason will only be what the admin has marked of the checkboxes that have pre-written reasons.
3. If the admin doesn't write anything and doesn't mark any checkboxes, then it will fallback to the generic reason (Breaking Rules).

With these improvements, we can solve the ugly ASLAY reason text when nothing is written and also make the custom reason field a better experience overall.

Something to note: currently, if an admin does not want to put the custom reason he wrote, he could just unmark the checkbox and it will not be there. If the improvements are made, this would no longer be possible. To remedy this, an X could be added to quickly clear any typed text.

Thanks for your attention.

[wgaylord]

There is a config options to make the Custom Reason check box checked by default already btw.

[Tiagoquix]

Sure, but that creates the ugly reason problem. That's why I suggested some new changes. :)