-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[4.0] Actions logs toolbar #24562
[4.0] Actions logs toolbar #24562
Conversation
All components only have the options and help on the right hand side this PR brings the action logs component into sync
@@ -98,12 +98,12 @@ protected function addToolbar() | |||
{ | |||
JToolbarHelper::title(JText::_('COM_ACTIONLOGS_MANAGER_USERLOGS'), 'list-2'); | |||
|
|||
JToolbarHelper::deleteList('JGLOBAL_CONFIRM_DELETE', 'actionlogs.delete'); | |||
JToolBarHelper::custom('actionlogs.exportSelectedLogs', 'download', '', 'COM_ACTIONLOGS_EXPORT_CSV', true); | |||
JToolBarHelper::custom('actionlogs.exportLogs', 'download', '', 'COM_ACTIONLOGS_EXPORT_ALL_CSV', false); JToolbarHelper::deleteList('JGLOBAL_CONFIRM_DELETE', 'actionlogs.delete'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JToolBarHelper::custom('actionlogs.exportLogs', 'download', '', 'COM_ACTIONLOGS_EXPORT_ALL_CSV', false); JToolbarHelper::deleteList('JGLOBAL_CONFIRM_DELETE', 'actionlogs.delete'); | |
JToolBarHelper::custom('actionlogs.exportLogs', 'download', '', 'COM_ACTIONLOGS_EXPORT_ALL_CSV', false); | |
JToolbarHelper::deleteList('JGLOBAL_CONFIRM_DELETE', 'actionlogs.delete'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No this was deliberate to put the delete actions at the end of the list see Banners/tracks as an example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
coding style: This is only moving JToolbarHelper::deleteList('JGLOBAL_CONFIRM_DELETE', 'actionlogs.delete');
to the next line and not be on the same line 102.
administrator/components/com_actionlogs/views/actionlogs/view.html.php
Outdated
Show resolved
Hide resolved
…html.php Co-Authored-By: brianteeman <brian@teeman.net>
You could use Joomla\CMS\Toolbar\Toolbar; for Toolbar like in other views. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24562. |
I deliberately didn't do any namespacing here as it makes sense to do all of that in its own pr. Single purpose pr are much easier for people to test. |
I have tested this item ✅ successfully on 4236002 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24562. |
1 similar comment
I have tested this item ✅ successfully on 4236002 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24562. |
Status "Ready To Commit". |
Thanks! |
thanks |
All components only have the options and help on the right hand side this PR brings the action logs component into sync
Before
After