[#20212] - [4.0] Transition of Joomla's Wrapper module and component from HTML4 to HTML5
- Closed
- 9 Aug 2018
- Medium
- Build: 4.0-dev
- # 20212
- Diff
- BaleshSrle:4.0-dev
User tests: Successful: 0 Unsuccessful: 0
Pull Request for Issue # .
Summary of Changes
Made a transition in Joomla's WRAPPER Module and Component from HTML4 to HTML5 standard.
Testing Instructions
Using "px" or "%" in "height" field (required by HTML5 standard).
If scrollbars do appear after selecting "No" option in "Scroll Bars" while creating wrapper element, you need to change the dimensions of wrapper element (ie. height and/or width of wrapper element).
Expected result
To work flawlessly.
Documentation Changes Required
Need to modify 6 files to work without any problems or hiccups:
-
modules/mod_wrapper/mod_wrapper.xml
-
modules/mod_wrapper/mod_wrapper.php
-
modules/mod_wrapper/Helper/WrapperHelper.php
-
modules/mod_wrapper/tmpl/default.php
-
components/com_wrapper/tmpl/wrapper/default.xml
-
components/com_wrapper/tmpl/wrapper/default.php
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_wrapper Modules |
| Labels |
Added:
?
|
||
merge it and I will update the other PR
| Title |
|
||||||
We need to take care here on upgrades, because after the upgrade the width and height are still added in numbers only without px. So either in the code we do some checks or an upgrade script should be added. Like it is now we have a BC break we can avoid.
@BaleshSrle are you willing to do the update scripts that we can test it and merge?
franz-wohlkoenig
- | Status | Pending | ⇒ | Information Required |
| Status | Information Required | ⇒ | Pending |
If scrollbars do appear after selecting "No" option in "Scroll Bars" while creating wrapper element, you need to change the dimensions of wrapper element (ie. height and/or width of wrapper element).
Because the width of the scrollbars vary from browser to browser I don't think that this instruction is acceptable for users. A No should be a No without otherwise necessary fine tunings.
As discussed earlier in other issues/prs you should additionally set the HTML-4 attribute "scrollbar=no" as fallback (like the rest of the world does, too) as long as there is no CSS solution or HTML5 solution for all supported browsers provided.
In the current stage I would not merge it. If @BaleshSrle is willing to update it, then I would leave it open.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-08-09 05:24:23 |
| Closed_By | ⇒ | laoneo |
@dgrammatiko can this work together with #18916?