Pending

User tests: Successful: Unsuccessful:

avatar ventsonge
ventsonge
14 Jun 2019

Pull Request for Issue #25194 .

Summary of Changes

Remove the append of template at the end of the form since it's unnecessary.
add the display none to the template tag, cause it's not properly recognize on IE

Testing Instructions

Create a subform and try to display it on IE (tested with 11.0)

avatar ventsonge ventsonge - open - 14 Jun 2019
avatar ventsonge ventsonge - change - 14 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jun 2019
Category Administration Templates (admin) Layout JavaScript
avatar Quy
Quy - comment - 14 Jun 2019

Pinging @Fedik

avatar Fedik
Fedik - comment - 14 Jun 2019

just make it display:none, inline, that should be enough, and without overhead

<template class="subform-repeatable-template-section" style="display:none">
...
</template>
avatar Quy Quy - change - 24 Jun 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-06-24 20:39:52
Closed_By Quy
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jun 2019
Closed_By Quy joomla-cms-bot
avatar joomla-cms-bot
joomla-cms-bot - comment - 24 Jun 2019

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/25200

avatar joomla-cms-bot joomla-cms-bot - close - 24 Jun 2019
avatar Quy
Quy - comment - 24 Jun 2019

Please test PR #25321


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

Add a Comment

Login with GitHub to post a comment