Bug 107336

Summary:	[testcase] nobr does not work with tables set to display:inline
Product:	[Applications] konqueror	Reporter:	rgpublic
Component:	khtml renderer	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED WORKSFORME
Severity:	normal	CC:	aiacovitti, finex
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Testcase

Description rgpublic 2005-06-13 16:59:54 UTC

Version:            (using KDE KDE 3.4.1)
Installed from:    SuSE RPMs

If I set a table to display:inline and use nobr to
keep text one the same line the nobr is ignored.
I'll attach a testcase.
Please compare rendering in Moz/IE and Konq.

Comment 1 rgpublic 2005-06-13 17:00:29 UTC

Created attachment 11429 [details]
Testcase

Comment 2 lexual 2005-07-11 10:32:07 UTC

confirming for 3.4.1

Comment 3 Allan Sandfeld 2006-11-13 01:35:42 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.

Comment 4 Allan Sandfeld 2006-11-13 02:43:39 UTC

We the patch the second case now works. The first case still break for some reason

Comment 5 FiNeX 2008-04-07 00:27:38 UTC

Comment #4 confirmed even on konqueror 4.0.3.

Comment 6 Andrea Iacovitti 2012-05-26 23:01:51 UTC

Can not reproduce with KDE 4.8.3