Project

General

Profile

Actions

Task #47003

closed

Epic #55575: Streamline JavaScript Code in the TYPO3 Backend

Replace old DynTabMenu code with jQuery/RequireJS plugin

Added by Benni Mack almost 11 years ago. Updated about 9 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2013-04-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.1
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

On the JS code sprint, the task to work on the DynTabMenu to be abstracted and
cleaned is done via RequireJS + jQuery.

A completely new jQuery plugin is written that stores the current selected tab ID
in the local storage (available in IE8+) or falls back to the first tab of the menu
item.

The changes separates logic (JS) from structure (HTML), and removes
plain old JS.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #62897: RTE: Change to tab menu div id breaks RTE load in IRRE and other nested elementsClosed2014-11-12

Actions
Has duplicate TYPO3 Core - Bug #47942: Move and change tab.jsClosed2013-05-05

Actions
Actions #1

Updated by Benni Mack almost 11 years ago

  • Tracker changed from Bug to Task
Actions #2

Updated by Gerrit Code Review almost 11 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19708

Actions #3

Updated by Gerrit Code Review over 10 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19708

Actions #4

Updated by Gerrit Code Review over 10 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19708

Actions #5

Updated by Gerrit Code Review about 10 years ago

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

Actions #6

Updated by Gerrit Code Review about 10 years ago

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

Actions #7

Updated by Daniel Siepmann about 10 years ago

  • Parent task set to #55575
Actions #8

Updated by Gerrit Code Review about 10 years ago

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

Actions #9

Updated by Gerrit Code Review over 9 years ago

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

Actions #10

Updated by Gerrit Code Review over 9 years ago

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

Actions #11

Updated by Gerrit Code Review over 9 years ago

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

Actions #12

Updated by Gerrit Code Review over 9 years ago

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

Actions #13

Updated by Gerrit Code Review over 9 years ago

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

Actions #14

Updated by Gerrit Code Review over 9 years ago

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

Actions #15

Updated by Benni Mack over 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #16

Updated by Stanislas Rolland over 9 years ago

  • Status changed from Resolved to New

Following this change, FormEngine->dynNestedStack is incorrect. It seems that the numbering of tabs has changed. But there are also sometimes more that one element at the same level in the stack.

Therefore the RTE cannot display correctly in a tab. Most obviously, the height of the iframe is wrong as well as the width of ExtJS comboboxes. This is because browsers cannot calculate and set dimensions inside an element with display: none.

The problem does not appear outside a tab. For example, when using the full screen RTE wizard.

The problem does not appear in IREE elements, it they are not inside a tab.

The problem occurs also in flexforms that are rendered with tabs containing RTE-enabled fields.

Actions #17

Updated by Gerrit Code Review about 9 years ago

  • Status changed from New 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 http://review.typo3.org/36401

Actions #18

Updated by Benni Mack about 9 years ago

  • Status changed from Under Review to Rejected

solved differently now.

Actions

Also available in: Atom PDF