Bug 136240

Summary:	In squirrelmail + icons plugin, images are displayed vertically instead of horizontally (like in other browsers)
Product:	[Applications] konqueror	Reporter:	Santiago Romero <sromero>
Component:	khtml	Assignee:	Konqueror Developers <konq-bugs>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

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.