Closed Bug 954731 Opened 10 years ago Closed 10 years ago

Remove explicit fonts from default message styles

Categories

(Instantbird Graveyard :: Conversation, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aleth, Assigned: aleth)

References

Details

Attachments

(1 file)

*** Original post on bio 1299 at 2012-02-28 21:27:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***
Blocks: 954509
Attached patch PatchSplinter Review
*** Original post on bio 1299 as attmnt 1207 at 2012-02-28 21:27:00 UTC ***

Removes explicit fonts from the css (so that font selection
via IB Preferences works).

Separated out from bug 954509 (bio 1074) (thanks Mic)
Attachment #8352958 - Flags: review?(clokep)
Comment on attachment 8352958 [details] [diff] [review]
Patch

*** Original change on bio 1299 attmnt 1207 at 2012-03-04 16:51:30 UTC ***

This looks fine, and is wanted according to flo:

(In reply to bug 954509 (bio 1074), comment #11)
> (In reply to bug 954509 (bio 1074), comment #6)
> 
> > * I would like to understand why "Sans" was listed in the font-family rules
> > that are being removed before r+ing this.
> 
> I discussed this with idechix, and it seems using the OS default sans-serif
> font is a good way forward for Linux.
Attachment #8352958 - Flags: review?(clokep) → review+
Assignee: nobody → aleth
Status: NEW → ASSIGNED
*** Original post on bio 1299 at 2012-03-07 00:29:57 UTC ***

Checked in as http://hg.instantbird.org/instantbird/rev/5540a627ace2

Thanks for fixing this!
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.2
You need to log in before you can comment on or make changes to this bug.