Bug 115909

Summary: Changing font size in www.freebsd.org screws up layout
Product: [Applications] konqueror Reporter: Hasso Tepper <hasso>
Component: khtmlAssignee: Konqueror Developers <konq-bugs>
Status: RESOLVED WORKSFORME    
Severity: normal CC: james
Priority: NOR    
Version: 3.4.92   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:

Description Hasso Tepper 2005-11-08 11:44:33 UTC 
Version:           3.4.92 (using KDE 3.4.92 (beta2, >= 20051010), compiled sources)
Compiler:          Target: i486-linux-gnu
OS:                Linux (i686) release 2.6.13.4-rt14

You can change font size on www.freebsd.org by clicking "Normal / Large" links in upper right corner of the page. But it screws up layout of the page.
Comment 1 James Spahlinger 2008-04-19 13:25:31 UTC 
Things do get a tad out of whack when you click that button. The "Foundation" link gets bumped to the next line. Some of the text falls outside of the colored boxes ss well. I'm not sure how "broken" the original reporter's original report was referring to got 2 years ago.

Comparing the behavior to mozilla Firefox, konqueror does not handle this correctly.

Using Gentoo Linux ~x86 (testing in gentoo lingo). KDE 4.0.3 and the majority of packages compiled using gcc 4.3.0.
Comment 2 Philip Rodrigues 2008-04-20 00:00:56 UTC 
This one is fun :-).

The underlying bug is that konqueror 3.5 doesn't reload/relayout the page correctly when the stylesheet is changed via javascript - this leads to the formatting of the header bar getting "screwed up". It appears to be a javascript issue, since you can change the stylesheet from view->use stylesheet, and the reloading/relayouting is done correctly.

Now, the site developers noticed this, and have a workaround in their javascript style switcher code:

window.onload = function (e)
{
  if (is_konq) {
    document.getElementById("searchnavlist").getElementsByTagName("li")[0].style.display = 'none';
    return;
  }
...
}

ie, for konqueror, it tries to hide the normal/large links. Unfortunately, the id of the element it wants is "SEARCHNAVLIST" in caps, so this call fails. I don't know what the correct behaviour is - should getElementById be case-sensitive?

So the site workaround for the konq bug has a bug itself...

Anyway, it's all kinda moot because the site works fine in trunk - the normal/large link is shown, and correctly resizes the fonts on the page.
Comment 3 James Spahlinger 2008-04-25 08:47:03 UTC 
Closing as works for me as this works in trunk.