? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
12 May 2019

Many of the toolbar buttons use a regular button layout but the majority are using the joomla-toolbar-button element

For consistency and to make styling easier this changes the batch button to use the popup layout instead of the batch layout so that the button uses the custom element.

Partial fix for #24850

avatar brianteeman brianteeman - open - 12 May 2019
avatar brianteeman brianteeman - change - 12 May 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 May 2019
Category Libraries
avatar wilsonge
wilsonge - comment - 13 May 2019

From a quick code comparison are we covering the batch button being disabled when no items are selected (i.e. the onclick attribute we have here https://github.com/joomla/joomla-cms/blob/4.0-dev/layouts/joomla/toolbar/batch.php#L26)

avatar brianteeman
brianteeman - comment - 13 May 2019

yes seems like it

batch

avatar wilsonge wilsonge - change - 13 May 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-13 10:06:41
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 13 May 2019
avatar wilsonge wilsonge - merge - 13 May 2019
avatar wilsonge
wilsonge - comment - 13 May 2019

Perfect - thanks!

avatar brianteeman
brianteeman - comment - 13 May 2019

Thanks

Add a Comment

Login with GitHub to post a comment