Bug 136240 - In squirrelmail + icons plugin, images are displayed vertically instead of horizontally (like in other browsers)
Summary: In squirrelmail + icons plugin, images are displayed vertically instead of ho...
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-24 18:41 UTC by Santiago Romero
Modified: 2006-11-13 01:35 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Santiago Romero 2006-10-24 18:41:53 UTC
Version:            (using KDE KDE 3.5.4)
Installed from:    Debian testing/unstable Packages
OS:                Linux

I've noticed that konqueror renders the "squirrelmail webmail" main menu bar in
a different way (vertically instead of horizontally) than ALL the other navigators do.

 Just test this URL in konqueror and other browsers:

http://www.sromero.org/test/test.html

 (It's not my code, it's part of squirrelmail webmail).
Comment 1 Maksim Orlovich 2006-10-24 19:02:12 UTC
That's some broken markup... (<table style="display:inline;"?, unclosed onclick, etc..)
Comment 2 Santiago Romero 2006-10-25 10:19:53 UTC
 Well... the w3c validator does not say anything about broken markup, just about the TABLEs inside the document, but the fact is that ALL THE OTHER BROWSERS (Explorer 5, Explorer 6, Explorer 7 beta, Firefox 1.0.5, Firefox 2.0, and Opera) render the icons correctly, and Konqueror does not.

 It seems konqueror's fault (at least that's what people will think).

 Any idea? :?
Comment 3 Allan Sandfeld 2006-11-13 01:35:35 UTC
SVN commit 604496 by carewolf:

Rewrite display:inline on TABLE to display:inline-table
A quirk Dirk left out ages ago, but it looks like we need it
BUG: 136240
BUG: 129671
CCBUG: 107336


 M  +2 -2      cssstyleselector.cpp  


--- branches/KDE/3.5/kdelibs/khtml/css/cssstyleselector.cpp #604495:604496
@@ -633,8 +633,8 @@
                 style->setDisplay(TABLE_CELL);
                 style->setFloating(FNONE);
             }
-//             else if (e->id() == ID_TABLE)
-//                 style->setDisplay(style->isDisplayInlineType() ? INLINE_TABLE : TABLE);
+            else if (e->id() == ID_TABLE)
+                style->setDisplay(style->isDisplayInlineType() ? INLINE_TABLE : TABLE);
         }
 
         // Table headers with a text-align of auto will change the text-align to center.