?
avatar infograf768
infograf768
7 Jul 2020

Summary of Changes

Correcting wrong body padding.
Therefore correcting wrong Close button padding and margin to display as square
Both body and close button will now display as in back-end

Testing Instructions

Display any modal in frontend, for example through CMS Content when editing an article OR editing Template settings and Select Image

Actual result BEFORE applying this Pull Request

Example below is for Image Select in Template settings
Screen Shot 2020-07-07 at 08 17 22

Close button detail
Screen Shot 2020-07-07 at 08 59 25

Expected result AFTER applying this Pull Request

Image below is for CMS Content =>Image
Screen Shot 2020-07-07 at 08 56 56

Close button detail
Screen Shot 2020-07-07 at 08 41 13

avatar infograf768 infograf768 - open - 7 Jul 2020
avatar Quy
Quy - comment - 9 Jul 2020

Tested successfully.

avatar astridx
astridx - comment - 10 Jul 2020

I applyed the code manually, run npm ci and tested successfully. Header in modal is now fine.

avatar infograf768
infograf768 - comment - 10 Jul 2020

RTC done here as the PR is not present in issues.

avatar rdeutz rdeutz - change - 10 Jul 2020
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-10 09:25:08
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 10 Jul 2020
avatar rdeutz rdeutz - merge - 10 Jul 2020
avatar joomla-cms-bot joomla-cms-bot - change - 10 Jul 2020
Category Front End Templates (site) NPM Change
avatar PhilETaylor
PhilETaylor - comment - 25 Jul 2020

does this not then break article selection lists for example when selecting an article when creating a menu item? Cant test as on the road, but ALL modals could be affected by this change and so other types of modal needs to be checked too.

Add a Comment

Login with GitHub to post a comment