Skip to content
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

Add option to remove "clear" functionality from UI #489

Open
kiu opened this issue Oct 29, 2021 · 5 comments · May be fixed by #447
Open

Add option to remove "clear" functionality from UI #489

kiu opened this issue Oct 29, 2021 · 5 comments · May be fixed by #447

Comments

@kiu
Copy link

kiu commented Oct 29, 2021

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.

@rustysponge
Copy link

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".

@sj26
Copy link
Owner

sj26 commented Oct 29, 2021

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?

@rustysponge
Copy link

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...

@kiu
Copy link
Author

kiu commented Oct 30, 2021

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

@sj26 sj26 linked a pull request Nov 1, 2021 that will close this issue
@lthreva
Copy link

lthreva commented Jan 12, 2023

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,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants