489942 - Update fonts to be usable on WinMobile or readable on desktop

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

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

Attachment: patch

All themes currently use "Nokia Sans", the standard font for apps on Maemo/Hildon, and a size of 16.75pt, also from Maemo guidelines. This isn't very nice on desktop builds which fallback to any sans serif font.

It's worse on WinMobile which not only has no Nokia Sans, but 16.75pt isn't close to a good font size for many resolutions.

This patch adds Tahoma as a fallback for hildon, which is used for desktop builds. The patch also removes all font overrides on WinMobile, allowing the native fonts and sizes to kick in.

I also remove some unused hildon styles

First step on path to some WinMobile theme changes

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

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

>diff --git a/themes/hildon/platform.css b/themes/hildon/platform.css
>--- a/themes/hildon/platform.css
>+++ b/themes/hildon/platform.css
>@@ -39,29 +39,23 @@
> @import url(chrome://global/skin/);
> 
> /* hildon stuff ------------------------------------------------------------ */
>-popup,
>-panel,
>-menupopup {
>-  padding: 16px !important;
>-}
>-

Why are you deleting these rules?

Comment 2

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

(In reply to comment #1)

> >-popup,
> >-panel,
> >-menupopup {
> >-  padding: 16px !important;
> >-}
> >-
> 
> Why are you deleting these rules?

We don't use these anywhere, so I wanted to drop them. We can add stuff back when we work on a complete theme.

http://hg.mozilla.org/mobile-browser/rev/7990b5660d0c

Comment 3

[Dão Gottwald [:dao]]

bugspam