kiwix / kiwix-android

Kiwix for Android
https://android.kiwix.org
GNU General Public License v3.0
888 stars 446 forks source link

Change wording when deleting selected history items vs. all history items #2098

Closed Frans-Lukas closed 4 years ago

Frans-Lukas commented 4 years ago

Is your feature request related to a problem? Please describe.

When deleting history items from the history view, the confirmation dialog always says "Delete history?". I argue that this is not specific enough because there is a difference of deleting all items, or just the selected items which should be made more clear in the confirmation dialog.

Describe the solution you'd like

When trying to delete all items it should ask "Delete all history?"/"Clear all history?" and when trying to delete the selected history items it should ask "Delete selected history?"/"Delete 10 selected history items?"

Describe alternatives you've considered

Keep it as is as you can see that you have selected items, however it does not hurt to be too specific. I think some distinction should be made between the two dialogs.

Additional context

Current dialog text:

Deleting all items Screenshot_20200519-095949

Deleting selected items Screenshot_20200519-095957

macgills commented 4 years ago

"Delete 10 selected history items?"

That should be ruled out because we want to avoid pluralised strings eg "Delete 1 item" "Delete 2 items" as supporting this across all languages is a bit complicated.

"Delete all history?"/"Clear all history?"

This seems ambiguous to me because we delete the "currently viewed" history, ie all history if "showAllHistory" is toggled or "current book history" if it is not and to communicate that to a user I feel gets too wordy and more likely to confuse.

"Delete selected history?"

Is the only one that has my support right now but I could hear arguments

Frans-Lukas commented 4 years ago

I didn't know it only deleted the items you are currently viewing. I just assumed it deleted everything. Which I think is telling about the clarity of the dialog text.

What about "Delete history from Europa Universalis 4 Wiki?" or "Europe Universalis 4 Wiki: Delete history from this book?"?

and "Delete history from all books?" when all history is viewed.

macgills commented 4 years ago

only deleted the items you are currently viewing.

I am recalling this from memory, may be prudent to look at the code.

All the following seem acceptable to me

"Delete history from Europa Universalis 4 Wiki?"

"Delete history from all books?"

"Delete selected history?"