Bug 26717 – box-orient is not inherited

• Status: RESOLVED
• Resolution: WONTFIX
• Component: CSS

[Shinichiro Hamaji]

According to http://dev.w3.org/csswg/css3-flexbox/ , box-orient should be inherited, but they aren't inherited with current WebKit. I'll upload a test case and a patch.

Note that this spec seemed to change. As of August 2007, box-orient should NOT be inherited: http://www.w3.org/TR/2007/WD-css3-layout-20070809/#box-orient
I'm not sure if the current spec is fixed or tentative. If WebKit developers think we should wait until the spec is fixed, please feel free to mark this bug as won't fix.

[Shinichiro Hamaji]

Created attachment 31849 [details]
Test case

[Shinichiro Hamaji]

Created attachment 31850 [details]
Patch v1

 LayoutTests/ChangeLog                              |   20 +++++++++++++++
 LayoutTests/fast/flexbox/017.html                  |    6 ++++
 LayoutTests/fast/flexbox/019.html                  |    6 ++++
 LayoutTests/fast/flexbox/box-orient-inherited.html |   25 +++++++++++++++++++
 LayoutTests/fast/flexbox/flex-hang.html            |    2 +-
 .../flexbox/box-orient-inherited-expected.checksum |    1 +
 .../fast/flexbox/box-orient-inherited-expected.png |  Bin 0 -> 16746 bytes
 .../fast/flexbox/box-orient-inherited-expected.txt |   16 ++++++++++++
 WebCore/ChangeLog                                  |   26 ++++++++++++++++++++
 WebCore/rendering/style/RenderStyle.h              |    4 +-
 WebCore/rendering/style/StyleFlexibleBoxData.cpp   |    4 +--
 WebCore/rendering/style/StyleFlexibleBoxData.h     |    1 -
 WebCore/rendering/style/StyleRareInheritedData.cpp |    5 +++-
 WebCore/rendering/style/StyleRareInheritedData.h   |    3 +-
 14 files changed, 110 insertions(+), 9 deletions(-)

[Shinichiro Hamaji]

Created attachment 31851 [details]
expected png of my layout test

Please land this png file in LayoutTests/platform/mac/fast/flexbox as well.

[Shinichiro Hamaji]

Created attachment 31905 [details]
Patch v1

 LayoutTests/ChangeLog                              |   20 +++++++++++++++
 LayoutTests/fast/flexbox/017.html                  |    6 ++++
 LayoutTests/fast/flexbox/019.html                  |    6 ++++
 LayoutTests/fast/flexbox/box-orient-inherited.html |   25 +++++++++++++++++++
 LayoutTests/fast/flexbox/flex-hang.html            |    2 +-
 .../flexbox/box-orient-inherited-expected.checksum |    1 +
 .../fast/flexbox/box-orient-inherited-expected.png |  Bin 0 -> 16746 bytes
 .../fast/flexbox/box-orient-inherited-expected.txt |   16 ++++++++++++
 WebCore/ChangeLog                                  |   26 ++++++++++++++++++++
 WebCore/rendering/style/RenderStyle.h              |    4 +-
 WebCore/rendering/style/StyleFlexibleBoxData.cpp   |    4 +--
 WebCore/rendering/style/StyleFlexibleBoxData.h     |    1 -
 WebCore/rendering/style/StyleRareInheritedData.cpp |    5 +++-
 WebCore/rendering/style/StyleRareInheritedData.h   |    3 +-
 14 files changed, 110 insertions(+), 9 deletions(-)

[Shinichiro Hamaji]

Comment on attachment 31905 [details]
Patch v1

Added the expected image into the patch.

[Eric Seidel (no email)]

Hyatt?

[Dave Hyatt]

I suspect that's a typo... it's not clear to me why box-orient would be inherited.  Let me ask dbaron.

[Dave Hyatt]

Comment on attachment 31905 [details]
Patch v1

Clearing review flag for now while I try to determine why this changed.

[Shinichiro Hamaji]

It turned out this was just a typo of the spec.