? ? Pending

User tests: Successful: Unsuccessful:

avatar RickR2H
RickR2H
14 Jul 2021

Pull Request for Issue # .

Summary of Changes

The modules in Top-a, Top-b, Bottom-a, and Bottom-b all use margins to create spacing between the modules. This creates a indentation on the left and the right site of the module container. This PR removes the side margins and add the gutter the same like the Left and Right positions.

fix-margins

Testing Instructions

Add multiple modules to the Top and Bottom positions and see if the indentation is there. Apply the PR an rebuild the CSS.

Actual result BEFORE applying this Pull Request

Indentation on the left and right site of the module position.

Expected result AFTER applying this Pull Request

The indentation is removed.

Documentation Changes Required

No

avatar RickR2H RickR2H - open - 14 Jul 2021
avatar RickR2H RickR2H - change - 14 Jul 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jul 2021
Category Front End Templates (site) NPM Change
avatar sandewt sandewt - test_item - 14 Jul 2021 - Tested successfully
avatar sandewt
sandewt - comment - 14 Jul 2021

I have tested this item successfully on 6141952

See comment


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

avatar sandewt
sandewt - comment - 14 Jul 2021

Results after testing: OK

34377_after

avatar ricardo1709 ricardo1709 - test_item - 14 Jul 2021 - Tested successfully
avatar ricardo1709
ricardo1709 - comment - 14 Jul 2021

I have tested this item successfully on 6141952


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

avatar Quy Quy - change - 14 Jul 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 14 Jul 2021

RTC


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

avatar wilsonge wilsonge - close - 21 Jul 2021
avatar wilsonge wilsonge - merge - 21 Jul 2021
avatar wilsonge wilsonge - change - 21 Jul 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-07-21 21:53:43
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 21 Jul 2021

Thanks!

Add a Comment

Login with GitHub to post a comment