561097 - Disabled radio buttons are smaller than enabled radio buttons

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

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

The new web theme might have caused a regression here. Go to a page the enables/disables a group of radio buttons. Toggling the radios enabled state will cause the page layout to change.

I used the bugzilla "add attachment" page. Checking the "[ ] Patch" checkbox causes a group of radio buttons to change enabled state. The radios change size and cause the layout to change.

If this is because of a border size difference, we should be using a transparent border, wit the same width as the enabled state.

Comment 1

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

The problem came from a wrong padding for disabled "select input type="button". I've also change [disabled="true"] to [disabled] to match forms.css

Comment 2

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

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

pushed mobile-1.1:
http://hg.mozilla.org/releases/mobile-1.1/rev/501db1d3a8c8

Comment 3

[Aakash Desai [:aakashd]]

Is there a test page that I can verify against, Mark?

Comment 4

[Aakash Desai [:aakashd]]

Wow, -1 for me not reading the actual report thoroughly enough.

verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2.5pre) Gecko/20100426 Namoroka/3.6.5pre Fennec/1.1a2pre

and

Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:1.9.3a5pre) Gecko/20100426 Namoroka/3.7a5pre Fennec/1.1a2pre

Comment 5

[Aakash Desai [:aakashd]]

This should be a unit test.