Summary: | invalid display of table for which css style inline is defined | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Alexander Denisjuk <denisjuk> |
Component: | khtml renderer | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | j3nkl3r |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | table illustrating behaviour of vertical-align as applied to tables with display:inline |
Description
Alexander Denisjuk
2006-06-22 22:51:28 UTC
You can't set a table inline. You might want to try display: inline-table Of course you might argue that we should have the _quirk_ where we rewrite attempts to set tables inline to inline-table. I know Safari does this, but I am unsure how it is handled by other browsers. Why I can not set a table inline? I checked CSS by W3C CSS checker, it found it correct. 'inline-table' works fine, without linebreaks. However I noticed that Firefox displays 'inline-table' in incorrect way: with linebreak after. I tried 'inline'. Firefox displays table without linebreak, but Konqueror does. Since I use Konqueror, my wife prefers Firefox, I decided to report bugs to both browsers:-). Well actually if you interpret it strictly a TABLE with inline display cannot contain table-section or table-rows, because inline elements can only contain other inline elements or text. You need to have a table or inline-table to have something to put table-content into. Funny that Firefox supports inline-table only when declared quirky. Can you test with IE? Well, I agree with your argument, I will leave 'inline-table' in my html. So, may I submit it not as a bug, but as a wish for Konqueror team. I am still waiting for an account to submit a bug to Gecko/Firefox. I can not check with IE. I don't have any. 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. *** Bug 137505 has been marked as a duplicate of this bug. *** This is still a problem in 3.5.8. When may we expect to see the fix? Created attachment 22843 [details]
table illustrating behaviour of vertical-align as applied to tables with display:inline
Once display:inline is fixed, the next question is, does vertical-align behave properly? Please refer to the attached table, which is also available at http://www.zipcon.net/~swhite/docs/computers/browsers/table_vertical_align.html re#7: The testcase is broken. It bears a strict doctype but uses a quirk (display:inline is not the same as display:inline-table no matter how many browsers get it wrong).
Quirks, as logic demand, only work in quirkmode.
> the next question is, does vertical-align behave properly?
once the testcase is fixed, assuming the text in green does describe the proper behaviour, which I have no time to double-check, then it is all fine, with the possible exception of vertical-align: sub, which expected behaviour isn't described, but doesn't look consistent with what we are doing with sup.
|