Closed Bug 518926 Opened 10 years ago Closed 10 years ago

about:fennec page - text and rows to small

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
fennec1.0b5

People

(Reporter: madhava, Assigned: Gavin)

Details

(Whiteboard: [polish])

Attachments

(1 file)

The page looks like this on an n810 at the moment:
http://www.flickr.com/photos/madhava_work/3950910296/

we should increase the row-height to be 6-7mm, and increase the font size of the links so that it's more readable.
tracking-fennec: --- → ?
Whiteboard: [polish]
Attached patch patchSplinter Review
Forgot some changes to the wince theme in the firstrun bug (those rules got moved to header.css).

Adjusts padding on the anchors and list items to make links taller and keep the arrow positioned correctly.
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #405115 - Flags: review?(mark.finkle)
Also tested on the n810, and got Madhava's approval.
Comment on attachment 405115 [details] [diff] [review]
patch

>diff --git a/themes/wince/aboutPage.css b/themes/wince/aboutPage.css

> #aboutLogoContainer {
>   width: 300px;
> }
>@@ -61,20 +50,23 @@
>   border: 2px solid #e6e5e3;
>   font-size: 15px;

We might want to look at this too. I think we use "pt" on wince for now.
Attachment #405115 - Flags: review?(mark.finkle) → review+
Changed that to 8pt to match ".button-text" styling (hildon's style rule also matches its .button-text):

https://hg.mozilla.org/mobile-browser/rev/1e06ab280425
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → B5
verified FIXED on build:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091008
Fennec/1.0b4
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.