Project

General

Profile

Actions

Feature #89781

closed

Add data attributes to JavaScript files

Added by Patrick Schriner over 4 years ago. Updated 10 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2019-11-26
Due date:
% Done:

0%

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

Description

Add support for data attributes for JavaScript files.

This is mainly needed for allowing scripts like Cookiebot to interact with linked Javascript files.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #91499: Additional attributes for includeJS, includeCSS and all others include**Closed2020-05-26

Actions
Actions #1

Updated by Gerrit Code Review over 4 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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/62434

Actions #2

Updated by Gerrit Code Review over 4 years ago

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

Actions #3

Updated by Gerrit Code Review over 4 years ago

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

Actions #4

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/62434

Actions #5

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/62434

Actions #6

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/62434

Actions #7

Updated by Gerrit Code Review about 4 years ago

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

Actions #8

Updated by Gerrit Code Review about 4 years ago

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

Actions #9

Updated by Gerrit Code Review about 4 years ago

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

Actions #10

Updated by Gerrit Code Review about 4 years ago

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

Actions #11

Updated by Gerrit Code Review about 4 years ago

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

Actions #12

Updated by Gerrit Code Review about 4 years ago

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

Actions #13

Updated by Susanne Moog about 4 years ago

  • Status changed from Under Review to Accepted

As the review has been abandoned I'm setting the patch back to accepted. The way to go here would be to allow usage of the assetCollector for TypoScript included files (fluid and PHP are already possible).

Actions #14

Updated by Susanne Moog over 1 year ago

  • Sprint Focus set to On Location Sprint
Actions #15

Updated by Oliver Hader over 1 year ago

  • Sprint Focus deleted (On Location Sprint)
Actions #16

Updated by Patrick Schriner over 1 year ago

#91499 is the more generic solution which has already been merged.
Can someone please close this ticket?

Actions #17

Updated by Riccardo De Contardi 10 months ago

  • Related to Feature #91499: Additional attributes for includeJS, includeCSS and all others include** added
Actions #18

Updated by Riccardo De Contardi 10 months ago

  • Status changed from Accepted to Closed

Patrick Schriner Sorry for the late reply. Closing it

If you think that this is the wrong decision or experience the issue again please reopen it or open a new issue with a reference to this one. Thank you.

Actions

Also available in: Atom PDF