Task #25024
closed
Remove typo3/gfx
Added by Steffen Kamper over 13 years ago. Updated over 6 years ago.
100%
Description
These images are not used any longer. Also they are duplicated in sysext/t3skin/icons/gfx. Those which are missing there could be copied
to this place.
A rewrite rule in htaccess could be the backwards compatible way if
anyone refers direct to the old resources.
(issue imported from #M17569)
Updated by Mr. Hudson about 13 years ago
Patch set 1 of change I735f00af0c52d36f27fafa99c7c311f0969b16e9 has been pushed to the review server.
It is available at http://review.typo3.org/1985
Updated by Xavier Perseguers about 13 years ago
- Status changed from New to Under Review
- TYPO3 Version changed from 4.5 to 4.6
Updated by Xavier Perseguers about 13 years ago
- Tracker changed from Bug to Task
Updated by Xavier Perseguers about 13 years ago
- Target version changed from 1081 to 1238
Updated by Xavier Perseguers about 13 years ago
What would be the proper way for deprecating their use?
Updated by Georg Ringer about 13 years ago
htaccess to an ajax call to be able to log the call?
Updated by Georg Ringer almost 13 years ago
again: IMO this is a no go to just remove things in core. Everything is deprecated and so must also be the icons.
i searched for typo3/gfx/ and IMO the admin panel uses it:
$outTable.= '<tr>' .
'<td style="white-space:nowrap;"><img src="typo3/gfx/clear.gif" width="' . (($depth + 1 - $row[2]) * 18) . '" height="1" alt="" /><img ' .
t3lib_iconWorks::skinImg(TYPO3_mainDir, 'gfx/i/pages.gif', 'width="18" height="16"') . ' align="top" border="0" alt="" />' . $row[1] .
'</td><td><img src="typo3/gfx/clear.gif" width="10" height="1" alt="" /></td><td>...</td></tr>';
and install tool
} else die('No typo3/gfx/typo3logo.gif file!');
and statictemplates uses it in a sql insert.
furthermore: did you do a check if all icons are available in t3skin? IMO there is a fallback from there to typo3/gfx/?
Updated by Oliver Hader almost 13 years ago
- Target version changed from 1238 to 1281
Updated by Thorsten Kahler almost 13 years ago
If the files are still in use, we can change that.
As with any other stuff IMHO we have a deprecation for those files also, don't we? So redirecting via an (optional) logger script might be a valid step before removing the files without proper compensation.
Updated by Xavier Perseguers almost 13 years ago
- Target version changed from 1281 to 4.6.0-beta1
Updated by Xavier Perseguers almost 13 years ago
- Priority changed from Should have to Must have
Updated by Xavier Perseguers almost 13 years ago
- Target version deleted (
4.6.0-beta1)
Updated by Xavier Perseguers over 12 years ago
- Status changed from Under Review to New
- Priority changed from Must have to Should have
- TYPO3 Version changed from 4.6 to 4.7
Updated by Felix Kopp about 11 years ago
Anything new? How can the unused icons be identified?
gfx/selicons
are used for example when t3skin is inactive.
Updated by Riccardo De Contardi over 9 years ago
Searching for "typo3/gfx" inside TYPO3 6.2.9 core brings the following results:
/typo3/sysext/css_styled_content/static/setup.txt
/typo3/sysext/css_styled_content/static/v6.0/setup.txt
/typo3/sysext/css_styled_content/static/v6.1/setup.txt
/typo3/sysext/frontend/Classes/View/AdminPanelView.php
/typo3/sysext/indexed_search/Classes/Controller/SearchController.php
/typo3/sysext/indexed_search/Classes/Controller/SearchFormController.php
Updated by Riccardo De Contardi almost 9 years ago
same search, results for:
1. TYPO3 6.2.13:
/ChangeLog /typo3/sysext/css_styled_content/static/setup.txt /typo3/sysext/css_styled_content/static/v6.0/setup.txt /typo3/sysext/css_styled_content/static/v6.1/setup.txt /typo3/sysext/frontend/Classes/View/AdminPanelView.php /typo3/sysext/indexed_search/Classes/Controller/SearchController.php /typo3/sysext/indexed_search/Classes/Controller/SearchFormController.php
2. TYPO3 7 (latest master):
/ChangeLog /typo3/sysext/core/Documentation/Changelog/7.1/Breaking-64143-FlagFilesMoved.rst /typo3/sysext/core/Documentation/Changelog/7.1/Breaking-64143-FlagFilesMoved.rst /typo3/sysext/core/Documentation/Changelog/7.1/Breaking-64143-FlagFilesMoved.rst /typo3/sysext/frontend/Classes/View/AdminPanelView.php
Updated by Wouter Wolters almost 9 years ago
There are many more places where images are still used from this folder.
For example I wrote https://review.typo3.org/#/c/40660/ that moves a few images
Updated by Benni Mack almost 9 years ago
- Target version set to 7.4 (Backend)
- Sprint Focus set to Stabilization Sprint
Updated by Wouter Wolters almost 9 years ago
- Target version changed from 7.4 (Backend) to 7.5
- Sprint Focus deleted (
Stabilization Sprint)
Let's kick this in 7.5..
Updated by Gerrit Code Review almost 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/42299
Updated by Gerrit Code Review almost 9 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/42299
Updated by Benni Mack almost 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 9d192760155f455c42efb376b552a0cd94598215.
Updated by Riccardo De Contardi over 6 years ago
- Status changed from Resolved to Closed