Closed Bug 1172049 Opened 9 years ago Closed 8 years ago

Reader mode does not respect sans/serif font family setting

Categories

(Toolkit :: Reader Mode, defect)

38 Branch
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1196675

People

(Reporter: mastabog, Unassigned)

Details

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Firefox/38.0
Build ID: 20150525141253

Steps to reproduce:

The Reader mode seems to force the usage of Georgia (for serif) and Helvetica (for sans) and does not respect the current fonts set by the user in the options.

In addition, it would be really good if the small Reader interface also allowed changing the font family and the page side margins. Personally I feel the text width is too short and the only way to change it is via the web developer, which has to be done every time Reader is invoked.

I'd like to get rid of the 3rd party "Reader" add-on but right now that allows me to change the font family and side margin size.
Summary: Reader mode does not respect sans/serif setting → Reader mode does not respect sans/serif font family setting
Apparently intentional behavior, according to bug 1139174.
Component: Untriaged → Reader Mode
OS: Unspecified → All
Product: Firefox → Toolkit
Hardware: Unspecified → All
(In reply to Mast from comment #0)
> User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0)
> Gecko/20100101 Firefox/38.0
> Build ID: 20150525141253
> 
> Steps to reproduce:
> 
> The Reader mode seems to force the usage of Georgia (for serif) and
> Helvetica (for sans) and does not respect the current fonts set by the user
> in the options.

Right. This is the same request as bug 1196675. Here's my reasons for closing that as WONTFIX:

(In reply to :Gijs Kruitbosch from comment #2)
> Here's my assumptions:
> 1) we can't change the defaults for the pref'd fonts everywhere because in
> some cases it breaks web pages
> 2) reader mode won't break if we change fonts
> 3) we want nice fonts in reader mode, and the defaults aren't very nice fonts
> 4) so in the case of default fonts being, err, the default, we want to
> override with 'nice' fonts
> 5) it's trivial to bypass the styling here with userContent.css or an add-on
> 
> 
> While it's theoretically possible to only do (4) here if the default font
> prefs are set to their defaults, that would take a non-trivial amount of
> code and be a pain to maintain/test (ie check both the 'override' and the
> 'respect user's choice of fonts' options). Maintaining the reader mode code,
> I don't think we want to do this provided there aren't very very very big
> reasons that somehow the fonts the user selects are sometimes Very
> Important, rather than just "Nicer" than the ones we've picked already, for
> a substantial subset of users.
> 
> As a result, wontfixing this.


> In addition, it would be really good if the small Reader interface also
> allowed changing the font family and the page side margins. Personally I
> feel the text width is too short and the only way to change it is via the
> web developer, which has to be done every time Reader is invoked.

Happily, adjusting the side margins can be done in Firefox 49 (currently developer edition, will be beta in 2-3 weeks, regular release a month or two after that).

As a workaround for you, I would suggest using the Stylish add-on or a userContent.css file, and use those to override the reader styling for serif/sans-serif fonts.
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.