? ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
8 Feb 2017

Pull Request for Issue # .

Summary of Changes

The template styles menu assignment tab uses span3 for each block. As these blocks can exceed 4 spans, this can give unpredicable results. This PR proposes using column-count instead. For the 2.5% of browsers not supporting column-count, blocks will appear in a single column.

Testing Instructions

Apply patch and navigate to the 'Menu Assignment' tab of a template style.

Before PR

menu-assign1

After PR

menu-assign2

Documentation Changes Required

None

avatar ciar4n ciar4n - open - 8 Feb 2017
avatar ciar4n ciar4n - change - 8 Feb 2017
Status New Pending
avatar ciar4n ciar4n - change - 8 Feb 2017
The description was changed
avatar ciar4n ciar4n - edited - 8 Feb 2017
avatar joomla-cms-bot joomla-cms-bot - change - 8 Feb 2017
Category Administration com_templates Templates (admin)
avatar C-Lodder
C-Lodder - comment - 8 Feb 2017

We can't use column-count in J3 as it's needs to support IE8

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 8 Feb 2017

I have tested this item successfully on 60be6fc


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 8 Feb 2017 - Tested successfully
avatar ciar4n
ciar4n - comment - 8 Feb 2017

@C-Lodder For IE8 and IE9 columns will appear in a single column which is not perfect. Currently using spans results in blocks floating to the left which means they could land anywhere considering the varied heights of each block. They will rarely appear in the correct order and will have random spacing.

avatar tuts-22
tuts-22 - comment - 8 Feb 2017

I have tested the pull request for issue #13971 and it works fine!!


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

avatar tuts-22
tuts-22 - comment - 8 Feb 2017

I have tested this item successfully on 60be6fc


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

avatar tuts-22 tuts-22 - test_item - 8 Feb 2017 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 11 Feb 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 11 Feb 2017

RTC


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

avatar rdeutz rdeutz - change - 12 Feb 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-12 15:50:01
Closed_By rdeutz
Labels Added: ? ?
avatar rdeutz rdeutz - close - 12 Feb 2017
avatar rdeutz rdeutz - merge - 12 Feb 2017

Add a Comment

Login with GitHub to post a comment