Fix for hidden attachments

[Priet]

I noticed that wasn't able to view all the attachments: depending on the browser, some attachments were hidden because of the single line for attachments.

Only in Firefox that lines grows if the attachments don't fit on the line.

In IE6 the line grows but the layout is a mess.

In IE7 the line doesn't grow and some attachment can't be seen.

I changed one line and added a new one in skins/default/mail.css:

Line number 591

Code: [Select]

#attachment-list
{
 margin: 0px;
 padding: 0px 0px 0px 68px;
 min-height: 18px;
 list-style-image: none;
 list-style-type: none;
 background: url(images/icons/attachment.png) 52px 1px no-repeat #DFDFDF;
 /* css hack for IE */
 height: expression(Math.min(18, parseInt(this.clientHeight))+'px');
}
Change it to: (see _height)

Code: [Select]

#attachment-list
{
 margin: 0px;
 padding: 0px 0px 0px 68px;
 min-height: 18px;
 list-style-image: none;
 list-style-type: none;
 background: url(images/icons/attachment.png) 52px 1px no-repeat #DFDFDF;
 /* css hack for IE */
 _height: expression(Math.min(18, parseInt(this.clientHeight))+'px'); /* Notice the _ before height, only affects IE6 */
}
Line number 613

Code: [Select]

#attachment-list li
{
 float: left;
 height: 18px;
 font-size: 11px;
 padding: 2px 10px 0px 10px;
}
Change it to: (see white-space)

Code: [Select]

#attachment-list li
{
 float: left;
 height: 18px;
 font-size: 11px;
 padding: 2px 10px 0px 10px;
 white-space: nowrap; /* Added so attachments are properly aligned, each on a single line if it doesn't fit on one line */
}
Changed a _height and added a white-space. That's all!

[seansan]

Please log your fix to bugtracker : http://trac.roundcube.net/newticket

[Priet]

Done:

http://trac.roundcube.net/ticket/1484894