Preface area background width is not 100% with horizontal scroll bar. Can be seen in both 6.x-1.9 and 6.x-2.x-dev, with any browser (I tried FF, IE 8, Google Chrome).

It can be observed when browser window is narrower, so horizontal scroll bar shows up. The scroll to the right edge.

See attached screenshot.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jwolf’s picture

Status: Active » Closed (duplicate)

This theme does not support screen resolutions below 1024x768

http://drupal.org/node/449968
http://drupal.org/node/339115

iva2k’s picture

Status: Closed (duplicate) » Needs review
FileSize
287 bytes

@jwolf:
Thanks for pointing out duplicates. I was not able to locate them using my keywords "preface width".
>This theme does not support screen resolutions below 1024x768
Understood. But I can't tell my website visitors to not reduce size of the window. Acquia_marina is the most solid theme out there. It outperforms Garland by miles. Garland breaks in so many ways. This bug is the only visual problem with AM that I encountered on 3 of websites I used it on. If it is fixed, it will make AM 100% solid.

Looking into this problem, I found out that there are "width: 960px" in style.css on some elements, but html and body widths are not set, so they are "width: 100%" by default. All browsers calculate 100% width of html and body to be width of actual window, ignoring width of included elements, which makes it less than 960px when window is narrow. (Yes, it is counter-intuitive and plain wrong. There are many illogical places in CSS implementations, but browsers can't/won't fix them because it will break 99% of all websites).

The simplest fix I found is single-liner "html { min-width: 975px; }". 975px is used instead of 960px because there is "padding-left: 15px" on one of the elements which adds to 960px of inner element.

Attached is a patch with the above fix. I verified that it fixes problem for IE8, FF3, Google Chrome. Please review for inclusion into 6.x-2.x-dev.

Regards,

iva2k’s picture

Here's how it looks after the patch.

jwolf’s picture

Assigned: Unassigned » jwolf

@iva2k - Thanks for the fix. I'll review and commit shortly.

tdh’s picture

Thank you iva2k! Now I will be able to use this theme again.

jwolf’s picture

Status: Needs review » Closed (won't fix)

Sorry, there has been a change of plans. We're in the process of porting the Acquia themes over to our new theme system, Fusion.

Therefore, we are unable to dedicate resources to fixing this in the 1.x branch since Fusion will solve this issue.

You can read more about Fusion at:

TNT 2.0 and the future of Drupal themes with Fusion
http://www.topnotchthemes.com/blog/090709/tnt-2-0-and-future-drupal-them...

Top Notch Themes pre-releases a landmark theme called Fusion
http://chrisshattuck.com/blog/top-notch-themes-pre-releases-landmark-the...

jwolf’s picture

Status: Closed (won't fix) » Active
jwolf’s picture

Status: Active » Fixed
sinasalek’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.