Bug 98563 - Form controls should always be horizontal
Summary: Form controls should always be horizontal
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tony Chang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-05 15:31 PDT by Tony Chang
Modified: 2012-10-05 16:57 PDT (History)
9 users (show)

See Also:


Attachments
Patch (57.20 KB, patch)
2012-10-05 15:40 PDT, Tony Chang
no flags Details | Formatted Diff | Diff
Patch (58.73 KB, patch)
2012-10-05 15:43 PDT, Tony Chang
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2012-10-05 15:31:38 PDT
WebKit doesn't support vertical controls yet.  It's a regression that we even try now.  The intent of the CSS in html.css is to force controls to horizontal-tb, but this must have been lost when switching from -webkit-block-flow to -webkit-writing-mode.
Comment 1 Tony Chang 2012-10-05 15:36:14 PDT
14:44 < tony^work> dhyatt: The user agent stylesheet (html.css) has some CSS 
                   that sets -webkit-block-flow: tb on form elements.  Should I 
                   just delete that code?
14:45 < dhyatt> oh actually it should turn into writing-mode
14:45 < dhyatt> patch rather than delete
14:45 < dhyatt> writing-mode should reset for form controls
14:45 < tony^work> The writing mode spec seems to suggest we should be able to 
                   support vertical controls
14:45 < dhyatt> just as many other props do
14:45 < dhyatt> yes, someday we might support vertical controls
14:46 < dhyatt> but until platforms have that support, we can't just mis-render
Comment 2 Tony Chang 2012-10-05 15:40:36 PDT
Created attachment 167398 [details]
Patch
Comment 3 Tony Chang 2012-10-05 15:43:15 PDT
Created attachment 167401 [details]
Patch
Comment 4 Tony Chang 2012-10-05 15:43:37 PDT
Comment on attachment 167401 [details]
Patch

I forgot to add the new -expected.html files.
Comment 5 WebKit Review Bot 2012-10-05 16:57:08 PDT
Comment on attachment 167401 [details]
Patch

Clearing flags on attachment: 167401

Committed r130569: <http://trac.webkit.org/changeset/130569>
Comment 6 WebKit Review Bot 2012-10-05 16:57:13 PDT
All reviewed patches have been landed.  Closing bug.