adjust style of form assistant/findbar buttons

VERIFIED FIXED

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: madhava, Assigned: vingtetun)

Tracking

({polish})

Details

(Whiteboard: [has patch][has review][needs approval])

Attachments

(3 attachments)

(Reporter)

Description

7 years ago
They don't really work right now.  See here:

http://www.flickr.com/photos/madhava_work/5476945747/in/photostream/

Getting new styles from Sean today.
(Reporter)

Comment 1

7 years ago
Actually, the field in the findbar shouldn't be transparent either.  You can get text showing through from below that looks like it's disabled gray text in the field.
tracking-fennec: ? → 2.0-
(Reporter)

Comment 2

7 years ago
From Sean via IRC:

mart3ll: vingtetun: http://grab.by/9kU8

mart3ll: same shape as now, same gradient as UI bg, arrows to 30% for disabled, stroke on buttons and field set at #383838

Comment 3

7 years ago
http://grab.by/9kU8

- same shape as now
- same gradient as UI bg
- arrows to 30% for disabled
- stroke on buttons and field set at #383838
Created attachment 517783 [details] [diff] [review]
Patch

Patched while bisecting!
Assignee: nobody → 21
Attachment #517783 - Flags: review?(mbrubeck)
Created attachment 517784 [details]
form assistant screenshot
Created attachment 517785 [details]
find helper screenshot
Comment on attachment 517783 [details] [diff] [review]
Patch

Looks nice!
Attachment #517783 - Flags: review?(mbrubeck) → review+
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [has patch][has review][needs approval]
Attachment #517783 - Flags: approval2.0?
Attachment #517783 - Flags: approval2.0? → approval2.0+
http://hg.mozilla.org/mobile-browser/rev/f211b1225fe8
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 9

7 years ago
VERIFIED FIXED on:

Build Id: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b13pre) Gecko/20110316 Firefox/4.0b13pre Fennec /4.0b6pre 

Devices: Motorola Droid 2(Android 2.2), Nokia N900 (Maemo GTK)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.