[TYPO3-dam-devel] RFC: #8598: Media > File: Icons are hidden by Fieldsize in IE 7.0

Dan Osipov dosipov at phillyburbs.com
Sat Jun 14 20:56:04 CEST 2008


I agree, I thought to provide a quick fix for the problem. Attached is 
the more complete solution, with accordance to the comments.

Dan Osipov
Calkins Media
http://phillyburbs.com/

Benjamin Mack wrote:
> Hey Dan,
> 
> nice! However I have one remark:
> 
> The code looks like this currently:
> =========
> $content = '
>     <!-- CONTROL PANEL: tx_dam:'.$item['uid'].' -->
> -    <div class="typo3-DBctrl">'.implode('', $actions).'</div>';
> +    '.implode('', $actions).'';
> }
> =========
> 
> However, I suggest make it shorter, by just having
> 
> $content = implode('', $actions);
> 
> as we don't need the comment, also the line before "compile items into a 
> DIV element, can go IMHO.
> 
> Can you make these small non-functional changes? Apart from that, it 
> looks good. I will have a look at it again, once the other patch 
> (styling) is improved as well again.
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 8598_2.patch
Type: text/x-diff
Size: 1216 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-dam/attachments/20080614/44fcc62b/attachment.patch 


More information about the TYPO3-team-dam mailing list