303095 - Feedview: store default stylesheet URL in externalCSS preference

Status: RESOLVED INVALID

(Firefox Graveyard :: RSS Discovery and Preview, enhancement)

Description

[Myk Melez [:myk] [@mykmelez]]

Currently we compose style from the default stylesheet and a user-supplied one,
if any.  This means users who want to create a completely new look for feeds
have to override all rules in the default stylesheet, while users who just want
to modify a subset of the look have to provide only the relevant rules.

We could instead store the default stylesheet value in the preference so we use
either the default stylesheet or a user-supplied one.  Then complete new lookers
wouldn't have to override all rules in the default, merely provide their own;
and subsetters would have to copy the default stylesheet rules into their
stylesheet.

The latter approach is better, since it involves no complex analysis of existing
rules.  The worst case is a file copy or cut and paste, whereas the worst case
for the current approach is carefully parsing the default stylesheet to
determine what every rule does and whether/how to override it, plus debugging
problems by examining the interaction of your rules and the default ones.

We should adopt the latter approach.  As a side-effect, it fixes bug 303049.

Comment 1

[:Gijs (he/him)]

Fixing summary so it's discoverable.

Comment 2

[Phil Ringnalda (:philor)]

Closing: Son of Feedview's "preview" display is going to be different enough to have brand new bugs.

Comment 3

[Tanner M. Young [:tmyoung]]

Resetting QA Contact to default.