Summary: | Remove a recipient row when the delete button is clicked | ||
---|---|---|---|
Product: | Claws Mail (GTK 2) | Reporter: | larivact |
Component: | UI/Compose Window | Assignee: | users |
Status: | REOPENED --- | ||
Severity: | enhancement | ||
Priority: | P3 | ||
Version: | other | ||
Hardware: | PC | ||
OS: | Linux |
Description
larivact
2017-01-10 18:27:52 UTC
The user can remove added recipients using the delete entry button which you mention. Since when? I am using 3.14.1 and it doesn't work for me. Is it fixed in the repo? What exactly doesn't work for you? When I hit the "Delete entry contents" button it only clears the field but the recipient row is still there. Yes, that's how it works. Is an empty row a problem for you? From a user experience perspective you should be able to. A row represents a recipient, to add a recipient you add a row, now what do you do to remove a recipient? The delete button only clears the field it isn't obvious that claws ignores empty recipient fields. However if the button removed the row it would be obvious that the recipient is removed. re-opened because it's now an enhancement request, even though I am in total disagreement with comment #6. (For instance, there is always an empty row.) Thanks for reopening I appreciate it.
>For instance, there is always an empty row.
Which is why the first row shouldn't have a delete button.
I apologize that would be bad design, instead the delete button should be disabled when it's the only row. Not exactly that simple: row 1 should get a greyed-out delete button IF it's the only recipient. If you have To: foo Cc: bar To: another You must be able to remove the 1st To:. Moreover, I wonder if a sweep is the best icon for 'delete' action, to me it's more clean than delete. But since it doesn't really delete the repicient but cleans the recipient field up, it's fine. Then don't call this a delete button. <<mad>> :-D (In reply to comment #6) > From a user experience perspective you should be able to. > A row represents a recipient, to add a recipient you add a row, > now what do you do to remove a recipient? A row is a space to enter recipients, can be zero, one or more than one. That association of one-row is one-recipient is completely imagined by you :) > The delete button only clears the field it isn't obvious that claws ignores > empty recipient fields. It isn't? Do you expect it to send a mail to a blank string "address"? I've heard a lot of absurd things, but this one is a real finisher :) > However if the button removed the row it would be > obvious that the recipient is removed. Could be *more* obvious, yes. Would also made replacement of addresses harder, because you have to go the end of upper field, and press enter, and select again the address header if incorrect, and then start typing. I guess the entry could be removed if the user clicks the delete button and the entry is already empty. That way current behaviour is preserved and doesn't disturb existing work-flows for address replacement, and you should be able to remove entries with a second click, as requested. (In reply to comment #10) [...] > Moreover, I wonder if a sweep is the best icon for 'delete' action, to me > it's more clean than delete. But since it doesn't really delete the > repicient but cleans the recipient field up, it's fine. Then don't call this > a delete button. > <<mad>> :-D It's default GTK stock icon, not ours. And yeah, you're reaching some kind of icon-caused insanity ;-) (In reply to comment #11) [...] > I guess the entry could be removed if the user clicks the delete button and > the entry is already empty. That way current behaviour is preserved and > doesn't disturb existing work-flows for address replacement, and you should > be able to remove entries with a second click, as requested. And while at it: after first click the brush icon on the button should be changed to a minus or delete red icon, to indicate that second click is going to remove the row completely. @wpp >Not exactly that simple: row 1 should get a greyed-out delete button IF it's the only recipient. That's exactly what I said in comment #9. >Then don't call this a delete button. Currently it isn't a delete button, this enhancement request is about making them delete buttons. @Ricardo >A row is a space to enter recipients, can be zero, one or more than one. Thanks, I didn't know that a row can contain multiple recipients. >Would also made replacement of addresses harder, because you have to go the end of upper field, and press enter, and select again the address header if incorrect, and then start typing. You can just: select the field, hit Ctrl+A followed by backspace. >I guess the entry could be removed if the user clicks the delete button and the entry is already empty. Buttons changing their functions is unintuitive. I still think these buttons should be delete buttons, but you Paul had a good argument that I didn't get yesterday. When you type into the field of the last row it immediately adds a new row below it. I think that instead there should be an add recipient button, with a simple plus as the icon. To make this system keyboard friendly backspace in an empty field could remove the row and enter in a field could create a new row below the current one. |