Multiple attachments cause email to be displayed incorrectly

[ryaner]

I just got a mail with 30odd attachments on it. This
caused RC to puch the email off to the right hand side
out of view of the mail window leaving me unable to
read the email. The list of attachments have also
exceeded the boundary which they are normally in.

[roundcube]

Logged In: YES 
user_id=1262041

CSS issue.

[thomasb]

Later... having 30 attachments is not very usual.

[kwach]

I think it should be a normal bug and for sure in to be done in stable release!
I had only three attachements, but with long names and email body was pushed out of the window.
Please resolve it to the stable release.

[jasondixon]

Screenshot of rendering issue

[jasondixon]

I attached a screenshot of this issue in action. I also found that Roundcube refused to output the attachment bar/filename at all if the filename exceeds 64 chars. This is probably more appropriate for another bug report, if at all.

[thomasb]

Any CSS experts out there?

[davo]

Putting an :after selector on #attachment-list seems to do the trick. Also, change #attachment-lists height property to min-height. Removing border-bottom is just a cosmetic thing because it puts a slight gap in the preview pane view and doesn't really add anything to the full message view.

--- skins/default/mail.css      (revision 508)
+++ skins/default/mail.css      (working copy)
@@ -657,13 +657,22 @@
 {
   margin: 0px;
   padding: 0px 0px 0px 68px;
-  height: 18px;
+  min-height: 18px;
   list-style-image: none;
   list-style-type: none;
-  background: url(images/icons/attachment.png) 52px 1px no-repeat #DFDFDF;
-  border-bottom: 1px solid #FFFFFF;
+  background: url(images/icons/attachment.png) 52px 1px no-repeat #DFDFDF;
 }

+#attachment-list:after
+{
+  content: ".";
+  display: block;
+  height: 0;
+  font-size:0;
+  clear: both;
+  visibility:hidden;
+}
+
 #attachment-list li
 {
 /*  display: block; */

[r@…]

Fix provided by davo solved the problem for me.

[thomasb]

It even works in IE7!
Applied patch to Trunk ([ba033aeb])