602893 - Put setting textboxes on separate row when in portrait

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

We easily run out of space for a label and textbox on the same row in Fennec preferences. This patch puts the label and textbox on separate rows.

The patch adds some CSS classes to the setting XBL to make it easier to access from CSS.

Comment 1

[Matt Brubeck (:mbrubeck)]

Comment on attachment 481859 [details] [diff] [review]
patch

Looks nice!  Just one nit: if you add flex="1" to the xul:textbox element in the setting-string XBL, then you can get delete this slow CSS rule:

>+#editor-setting[type="string"] .setting-input > textbox {
>   -moz-box-flex: 1;
> }

This one is important because it will be tested against every <textbox> element, and it contains a descendant selector.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #1)
> Comment on attachment 481859 [details] [diff] [review]
> patch
> 
> Looks nice!  Just one nit: if you add flex="1" to the xul:textbox element in
> the setting-string XBL, then you can get delete this slow CSS rule:
> 
> >+#editor-setting[type="string"] .setting-input > textbox {
> >   -moz-box-flex: 1;
> > }
> 
> This one is important because it will be tested against every <textbox>
> element, and it contains a descendant selector.

I think we should file a new bug on reworking about:config and config.css
I think we should be using vertical aligned textboxes there too. And I found a weird bug with the editor.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/624d369d9020

filed bug 602945

Comment 4

[Anna (Waverley)]

Can you provide some webpages to test this?