Ticket #1484633 (closed Bugs: fixed)

Opened 2 years ago

Last modified 16 months ago

Preview pane too wide in IE6

Reported by: seansan Owned by:
Priority: 5 Milestone: 0.2-stable
Component: User Interface Version: svn-trunk
Severity: minor Keywords:
Cc:

Description

The preview pane (or the mail message inside it) is a little too wide.

When previewing a message I would expect a vertical scrollbar, but not also a horizontal one to scroll the 10 remaining pixels to the right. This seems to be the case in every message that I preview. I have attached a screenshot.

Maybe the calculations for available width are incorrect? I could not find it to be set in the css file.

Attachments

untitled.PNG (3.5 KB) - added by seansan 2 years ago.
Example
RoundCubeLeopard.png (18.7 KB) - added by jarrod 2 years ago.
The same can be said for Safari under Mac OS X Leopard (10.5)
ie_message.r2122.patch (1.2 KB) - added by JohnDoh 16 months ago.
remove horizontal scroll from full message view in ie6

Change History

Changed 2 years ago by seansan

Example

Changed 2 years ago by thomasb

  • severity changed from normal to minor

Need some CSS-Hacks for IE6.

Changed 2 years ago by jarrod

The same can be said for Safari under Mac OS X Leopard (10.5)

Changed 2 years ago by till

Can't reproduce this: http://www.flickr.com/photos/till/2240904335/

Is IE6 still a plattform to deal with?

Changed 2 years ago by till

A friend of mine confirmed this working in Opera on Windows and Linux.

Changed 2 years ago by thomasb

Unable to fix this. Does anybody have a CSS solution for IE6?

Changed 2 years ago by seansan

The problem comes from the template watermark.html where a default width is set fot the loaded watermark of 245px. What should the width be?

- This accounts for the scrollbars on login (only seeing watermark)

Should the watermark be less wide than 245px?

- But then, all the messages that are loaded afterwards are still a little too wide. Again by the same width - maybe comes form elsewhere - The message headertable and remote images warning both have a offset of some px (left and right) making it fall out of the window again - and the scrollbar is back ;)

Should both these tables be a little less wide?

Suggestion is to replace

<div style="margin:10px auto; text-align:center">
<img src="images/rcube_watermark.png" width="245" height="245" alt="" />

with

<div style="margin:10px auto; text-align:center">
<img src="images/rcube_watermark.png" width="200" height="200" alt="" />

Changed 2 years ago by till

  • milestone changed from 0.1-stable to 0.1.1

Changed 22 months ago by thomasb

  • component changed from Interface to User Interface

Changed 19 months ago by alec

  • version changed from 0.1-rc2 to svn-trunk

From http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4852280

It's Internet Explorer frame bug

  When Internet Explorer displays a frameset document, if one of the 
  frames has too much vertical content to fit in its space on the screen, 
  as expected, a vertical scrollbar appears.  Unfortunately, IE considers
  the rectangular area *behind* the scrollbar as space for text layout. 
  The result is that the document lays out behind the scroll bar, and in 
  a stunning example of doublethink, IE then says "Whoops, the layout is 
  running off the right side of the page" and also displays a horizontal 
  scrollbar. Even resizing the frame manually won't get rid of the horizontal
  scrollbar (as long as the document still wants more vertical space than 
  is available to it).

Three possible solutions:
  1) Ignore the problem and wait for MS to fix the problem
     However, I could not find this bug on microsoft.com in knowledge base
     and don't know how to submit bugs to MS.
  2) Remove the DOCTYPE declaration from the top of the index.html file.
     The effect is apparently to throw IE into "quirks" mode, which 
     perversely seems to behave better in this case than "standards" mode.
  3) Turn on scrolling="yes" attribute, which forces the vertical (but not
     horizontal) scroll bar to appear and forces IE to realize its area
     is not available for text layout.  If all documents are longer than
     one screen, it won't matter that the vertical scroll bar is always
     present.  However, if any page fits fully on the screen, the scroll
     be will still be present (though the scroll arrows will not appear).

Changed 18 months ago by robin

I tried following:

--- program/steps/mail/func.inc (revision 1811)
+++ program/steps/mail/func.inc (working copy)
@@ -407,9 +407,10 @@
   $attrib_str = create_attrib_string($attrib, array('id', 'class', 'style', 'src', 'width', 'height', 'frameborder'));
   $framename = $attrib['id'];

-  $out = sprintf('<iframe name="%s"%s></iframe>'."\n",
+  $out = sprintf('<iframe name="%s"%s%s></iframe>'."\n",
          $framename,
-         $attrib_str);
+         $attrib_str,
+         strstr($_SERVER['HTTP_USER_AGENT'], 'MSIE 6') ? ' scrolling="yes"' : '');

   $OUTPUT->set_env('contentframe', $framename);
   $OUTPUT->set_env('blankpage', $attrib['src'] ? $OUTPUT->abs_url($attrib['src']) : 'program/blank.gif');

It hides the horizontal scroll bar, always shows the vertical scroll bar, and breaks display of HTML mail. So that's no option.

Changed 16 months ago by alec

  • status changed from new to closed
  • resolution set to fixed

Probably fixed in r2120.

Changed 16 months ago by tensor

Unfortunately, the bug is still present with r2120. Horizontal scrollbars still appear in IE6.

Changed 16 months ago by alec

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 16 months ago by JohnDoh

I have tried the CSS patch from r2120 in IE6 on 2 different computers and it definatly removed the horizontal scrollbars from preview pane for me. Also I have attached a patch that should fix the main message view in IE6 (agian it does for me on 2 different computers). Did you make sure you cleared your cache?

I have tried with both plain text and HTML emails if the patches still dont work could you give us an example message and I will see if i can figure it out. Thanks

Changed 16 months ago by JohnDoh

remove horizontal scroll from full message view in ie6

Changed 16 months ago by alec

  • status changed from reopened to closed
  • resolution set to fixed

Fixed in r2131.

Note: See TracTickets for help on using tickets.