Bug 107336 - [testcase] nobr does not work with tables set to display:inline
Summary: [testcase] nobr does not work with tables set to display:inline
Status: RESOLVED WORKSFORME
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-13 16:59 UTC by rgpublic
Modified: 2012-05-26 23:01 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Testcase (524 bytes, text/html)
2005-06-13 17:00 UTC, rgpublic
Details

Note You need to log in before you can comment on or make changes to this bug.
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