mailcatcher icon indicating copy to clipboard operation
mailcatcher copied to clipboard

Add option to remove "clear" functionality from UI

Open kiu opened this issue 2 years ago • 5 comments

We are using mailcatcher in a multi-user environment and sometimes people confuse the "clear" button with "clearing my search" and therefore kill all emails.

It would help if this button could be disabled (in combination with message limit) by a command line option.

kiu avatar Oct 29 '21 13:10 kiu

It could also help to move the button away from the search input in the UI and/or to change the label to "Clear all messages".

rustysponge avatar Oct 29 '21 13:10 rustysponge

Are you mainly wanting to prevent accidental usage?

The clear button does ask for confirmation with a pretty clear message:

You will lose all your received messages.

Are you sure you want to clear all messages?

sj26 avatar Oct 29 '21 21:10 sj26

Yes, but nevertheless it happens from time to time in our setup.

Therefore, in my opinion, a clearer message on the button and/or another button location would be helpful. I think the main problem is that material design (and others) suggests/uses to use a "clear this input" trailing icon on the right side of the input itself and that people are used to this pattern...

rustysponge avatar Oct 30 '21 07:10 rustysponge

I have an additional perspective to @rustysponge.

I believe its natural behaviour for a user when he has a task requiring him to use mailcatcher, that he clears all emails to have a fresh start with less clutter. That is cool in a single user environment.

In a multi user environment this user might delete all mails to declutter his experience, but having done this, might have deleted mails other users spent significant time on preparing scenarios for. Rather than solving this through processes (weekly memo: please everyone, do not clear the mailcatcher, think about the needs of your colleagues) a functionality preventing the user from doing so would solve that automatically.

Therefore from my perspective there are two parts. One is unintentional use of the clear functionality and the other is unintentional side effects of using the clear button.

For our org something like this would do the trick:

mailcatcher --messages-limit 500 --no-clear

kiu avatar Oct 30 '21 09:10 kiu

Hello,

We had exactly this same issue this morning: someone thought that the button was about clearing the search field. The person then clicked quickly the confirmation message.

Afterward, the person said that the confirmation message was indeed clear.

It seems that like other say in this issue, the "Clear" button could be moved or modified to be more precise.

I've seen that the linked pull request is no more in sync, how would you to move on this issue? Could I propose a PR with a rewording or a change in location of the button ( or search box)? It could be as easy as taking the same text for the HTML title and content of button, see code

Regards,

lthreva avatar Jan 12 '23 12:01 lthreva