[TYPO3-core] RFC #8990: Labels 'TEMP' and 'RECYCLER' are hardcoded
Oliver Klee
typo3-german-02 at oliverklee.de
Mon Jul 21 12:03:16 CEST 2008
Hi Stefano,
well done! +1 with the changes Bernhard suggested and the followings
nits changed:
> if ($title=='_recycler_') {
> $icon = 'gfx/i/recycler.gif';
> - $title='<b>RECYCLER</b>';
> + $title='<b>' . $GLOBALS['LANG']->getLL('recycler', true) . '</b>';
While you're changing this line, I propose to change the <b> to the
semantic <strong>. (That's your decision.)
> }
> - $icon = $theFile['writable'] ? 'gfx/i/_icon_'.$webpath.'folders_ro.gif' : $icon;
> + $icon = $theFile['writable'] ? 'gfx/i/_icon_' . $webpath . 'folders_ro.gif' : $icon;
>
> - return Array($title,$icon,$path);
> + return Array($title, $icon, $path);
Array -> array
> Index: typo3/sysext/lang/locallang_mod_file_list.xml
> ===================================================================
> --- typo3/sysext/lang/locallang_mod_file_list.xml (revision 3886)
> +++ typo3/sysext/lang/locallang_mod_file_list.xml (working copy)
> @@ -21,6 +21,8 @@
> <label index="c_rw">RW</label>
> <label index="c__REF_">Ref</label>
> <label index="files">files</label>
> + <label index="temp">TEMPORARY FILES</label>
> + <label index="recycler">RECYCLER</label>
I find this all-capitals style not very nice-looking. What about using
the opportunity and changing this to "Temporary files" and "Recycler"?
(That's also a matter of taste and your decision.)
Oli
More information about the TYPO3-team-core
mailing list