? Success

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
23 Aug 2016

Pull Request for Issue #11755

Summary of Changes

This PR fixes a media field lightbox issue when additional subforms are added extending the page height. This results in the lightbox background no longer filling the page. Setting the lightbox background div (#sbox-overlay) to 'fixed' ensures the background remains within the viewport regardless of the page height.

Testing Instructions

Create ../modules/mod_login/text.xml to which add the following...

<?xml version="1.0" encoding="UTF-8"?>
<form>
    <fieldset name="images" class="image-subform">
        <field name="image" type="media" default="" label="Label1"  description="Desc1" size="30" />
        <field name="text" type="textarea" filter="raw" rows="10" cols="42" default="" label="Label2" description="Desc2" />
    </fieldset>
</form>

Add the following to the mod_login.xml file:

<field name="subformtest" type="subform" formsource="modules/mod_login/text.xml" class="advancedSelect" min="1" max="50" multiple="true" layout="joomla.form.field.subform.repeatable" groupByFieldset="true" label="Image(s)" description="Add image(s) to display in gallery" />

Then in the Module Manager, open the Login module. Add a number of subforms extending the page height. Click 'Select' in the last subform.

Before patch

screen1

After Patch

screen2

avatar ciar4n ciar4n - open - 23 Aug 2016
avatar C-Lodder
C-Lodder - comment - 23 Aug 2016

Can't find this on JIssues, so marking this test as successful here

avatar zero-24
zero-24 - comment - 23 Aug 2016

Just reported it to jissues @c-lodder joomla/jissues#871

avatar zero-24 zero-24 - alter_testresult - 23 Aug 2016 - c-lodder: Tested successfully
avatar jeckodevelopment jeckodevelopment - test_item - 23 Aug 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 23 Aug 2016

I have tested this item successfully on 8843ffd


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

avatar zero-24 zero-24 - change - 23 Aug 2016
Status New Pending
avatar zero-24
zero-24 - comment - 23 Aug 2016

@C-Lodder i have just added you test


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

avatar zero-24 zero-24 - change - 23 Aug 2016
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 23 Aug 2016

RTC


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

avatar jeckodevelopment
jeckodevelopment - comment - 23 Aug 2016

bot is sleeping!

avatar mbabker
mbabker - comment - 23 Aug 2016

No it's not. We're fighting with ModSecurity yet again.

avatar wilsonge wilsonge - change - 24 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-24 22:16:27
Closed_By wilsonge
avatar wilsonge wilsonge - close - 24 Aug 2016
avatar wilsonge wilsonge - merge - 24 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 24 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 24 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment