?
avatar marcochirienti
marcochirienti
17 Mar 2017

The JToolbarHelper::deleteList method uses as default text the JTOOLBAR_DELETE language key.
The language key JTOOLBAR_DELETE is translated as "Delete".

In core components the language key used for deleteList buttons is JTOOLBAR_EMPTY_TRASH.
The language key JTOOLBAR_EMPTY_TRASH is translated as "Empty trash".

As a non-native English speaker, I think "Empty trash" is misleading.
It sounds to me like "permanently delete all items from the trash".
And that's not what the button actually do.

The button allows the user to delete selected items, so why don't we use the default "Delete" text?

avatar marcochirienti marcochirienti - open - 17 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 17 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 17 Mar 2017
avatar infograf768
infograf768 - comment - 17 Mar 2017

As far as I know we do have a difference.
We use JTOOLBAR_DELETE when there is no trash for the items i.e. deleting directly.
Example: administrator/index.php?option=com_installer&view=updatesites
or
administrator/index.php?option=com_languages&view=overrides

avatar marcochirienti
marcochirienti - comment - 17 Mar 2017

I think that the difference between items that can be trashed and items that can't be trashed is already clear.

For items that can be trashed, there is another button and is the "Trash" button (JToolbarHelper::trash).
The JToolbarHelper::trash method uses the JTOOLBAR_TRASH language key.
The language key JTOOLBAR_TRASH is translated as "Trash".

In core components the default behaviour in list views is to display only published and unpublished items (archived and trashed items are generally hidden).
If the state filter is set to "trashed" the list view displays the trashed items and the "Empty trash" button in the toolbar. Once an item has been trashed, can be permanently deleted.

What the "Empty trash" does is exactly what the "Delete" button does: permanently deletes an item or a list of items. It does not empty the "trash bin".

For this reason I assume they can have the same text.

avatar joomla-cms-bot joomla-cms-bot - edited - 17 Mar 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category UI/UX
avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Apr 2017
Category UI/UX Templates (admin) UI/UX
avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Apr 2017
Status New Discussion
avatar yasirunilan
yasirunilan - comment - 11 Apr 2017

For this one can we have the same text or a change is required?

avatar Quy
Quy - comment - 27 May 2017

PR #14954

avatar franz-wohlkoenig franz-wohlkoenig - change - 27 May 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-05-27 04:04:22
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 27 May 2017
Closed_Date 2017-05-27 04:04:22 2017-05-27 04:04:23
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 27 May 2017
avatar joomla-cms-bot
joomla-cms-bot - comment - 27 May 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 May 2017

closed as having PR #14954


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14703.

Add a Comment

Login with GitHub to post a comment