?
avatar C-Lodder
C-Lodder
6 Mar 2021

Steps to reproduce the issue

https://github.com/joomla/joomla-cms/blob/4.0-dev/components/com_wrapper/tmpl/wrapper/default.php#L36-L37

and

https://github.com/joomla/joomla-cms/blob/4.0-dev/modules/mod_wrapper/tmpl/default.php#L25-L26

Both the scrolling and frameborder are deprecated attributes, so I'd suggest removing these lines of code and also removing any associated parameters for them

avatar C-Lodder C-Lodder - open - 6 Mar 2021
avatar joomla-cms-bot joomla-cms-bot - change - 6 Mar 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Mar 2021
avatar richard67
richard67 - comment - 6 Mar 2021

@zero-24 As you made PR #32596 for the same iframe wrappers, do you think you can make another one for this issue here?

avatar zero-24
zero-24 - comment - 6 Mar 2021

Well removing that options can be done but should be independent from the PR mention.

avatar richard67
richard67 - comment - 6 Mar 2021

Yes, independently of course.

avatar Megh05
Megh05 - comment - 13 Mar 2021

@C-Lodder @richard67 I am interested to work on this issue. Thank you!

avatar richard67
richard67 - comment - 13 Mar 2021

@Megh05 Just to it, no need to ask for permission ;-)

avatar PhilETaylor
PhilETaylor - comment - 28 Mar 2021

Please test #32913

avatar richard67 richard67 - change - 28 Mar 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-03-28 20:58:22
Closed_By richard67
avatar richard67 richard67 - close - 28 Mar 2021
avatar richard67
richard67 - comment - 28 Mar 2021

Closing as having a pull request. Please test #32913 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment