Bug #58697

jQuery tabs pollution

Added by Robotic-Brain no-lastname-given about 1 year ago. Updated 7 months ago.

Status:Rejected Start date:2014-05-11
Priority:Should have Due date:
Assigned To:- % Done:

0%

Category:Extension Manager Spent time: -
Target version:-
TYPO3 Version:6.2 Is Regression:Yes
PHP Version: Sprint Focus:
Complexity:no-brainer

Description

The first list (<ul>) you output from class.ext_update.php gets converted to jQuery tabs.
This forces you to use tabs, even if you don't want to.
(or use this workaround:

$content .= '<script type="text/javascript">jQuery(function(){jQuery("#typo3-extension-configuration-forms").tabs("destroy");});</script>';
)

this can be easily fixed by removing the call to:

$("#typo3-extension-configuration-forms").tabs();

in: /typo3/sysext/extensionmanager/Resources/Public/JavaScript/main.js


Related issues

related to Core - Task #42197: Use central jQuery Resolved 2012-10-20
related to Core - Bug #43244: Support for class.ext_update.php missing in new EM in TYP... Resolved 2012-11-23

History

#1 Updated by Markus Klein about 1 year ago

  • Status changed from New to Accepted

Possible regression to: https://review.typo3.org/#/c/16780/8/typo3/sysext/extensionmanager/Resources/Private/Templates/UpdateScript/Show.html
or: https://review.typo3.org/#/c/15828/20/typo3/sysext/extensionmanager/Resources/Public/JavaScript/main.js

Either the markup is wrong (id attribute that should only be used with extension configuration forms) or the JS is wrong.

#2 Updated by Christian Kuhn about 1 year ago

  • Is Regression changed from No to Yes

#3 Updated by Gerrit Code Review about 1 year ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30647

#4 Updated by Gerrit Code Review 7 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/30647

#5 Updated by Stefan Neufeind 7 months ago

  • Status changed from Under Review to Rejected

refactored in CMS7

Also available in: Atom PDF