? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
28 May 2019

As far as I can tell this layout is only used administrator/index.php?option=com_banners&view=tracks for the export button.

If you look at the screenshots you will see that the export button is indented more than the new button.

image
image

This is because there is some css for the first button.

.subhead .btn-toolbar>:first-child { margin-left: 0; }

This should be applied to the export button as well but its not hence the indentation.

This is caused by the layout rendering the div for the modal before the button. This means that the button is no longer the first-child

This simple PR changes the order of the div and the button to resolve this.

After PR

image

NOTE any issues with the rendering of the modal are beyond the scope of this PR

avatar brianteeman brianteeman - open - 28 May 2019
avatar brianteeman brianteeman - change - 28 May 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 May 2019
Category Layout
avatar Quy
Quy - comment - 28 May 2019

I have tested this item successfully on 72b8f91


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

avatar Quy Quy - test_item - 28 May 2019 - Tested successfully
avatar pe7er
pe7er - comment - 29 May 2019

I have tested this item successfully on 72b8f91

The "Export" button on "Banners: Tracks" is now aligned the same as on the other views.


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

avatar pe7er pe7er - test_item - 29 May 2019 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 29 May 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 29 May 2019

Status "Ready To Commit".

avatar wilsonge wilsonge - change - 29 May 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-29 11:07:53
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 29 May 2019
avatar wilsonge wilsonge - merge - 29 May 2019
avatar wilsonge
wilsonge - comment - 29 May 2019

Thanks!

avatar brianteeman
brianteeman - comment - 29 May 2019

Thanks - took me far too long to work out the issue

Add a Comment

Login with GitHub to post a comment