Bug #58697

jQuery tabs pollution

Added by Robotic-Brain no-lastname-given almost 7 years ago. Updated about 6 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
Extension Manager
Target version:
-
Start date:
2014-05-11
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
Yes
Sprint Focus:

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 TYPO3 Core - Task #42197: Use central jQueryClosedStefan Neufeind2012-10-20

Actions
Related to TYPO3 Core - Bug #43244: Support for class.ext_update.php missing in new EM in TYPO3 6.0ClosedFrancois Suter2012-11-23

Actions
#1

Updated by Markus Klein almost 7 years 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 almost 7 years ago

  • Is Regression changed from No to Yes
#3

Updated by Gerrit Code Review over 6 years 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 about 6 years 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 about 6 years ago

  • Status changed from Under Review to Rejected

refactored in CMS7

Also available in: Atom PDF