Closed
Bug 877783
Opened 12 years ago
Closed 11 years ago
Switch Reader Mode title to bold for Clear Sans
Categories
(Firefox for Android Graveyard :: Reader View, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 930566
People
(Reporter: ibarlow, Assigned: marcosadp)
References
Details
(Whiteboard: [mentor=margaret][lang=css])
Attachments
(1 file)
730 bytes,
patch
|
Details | Diff | Splinter Review |
Using Clear Sans Light for Reader Mode article titles doesn't look that great. Thankfully bold looks good, so let's change our title styles accordingly.
http://cl.ly/image/0T0m2o003P20
Comment 1•12 years ago
|
||
To fix this, it looks like we should play around with the font-weight style here:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/themes/core/aboutReader.css#66
Assignee | ||
Comment 2•12 years ago
|
||
If this bug only consists of changing .header > h1 font-weight from 300 to bold in the aboutReader.css, I'm done and could submit a patch.
Updated•12 years ago
|
Assignee: nobody → marcosadp
Assignee | ||
Comment 3•12 years ago
|
||
Changes line 66 of /mobile/android/themes/core/aboutReader.css from font-weight: 300; to font-weight: bold;
Comment 4•12 years ago
|
||
Marcos it is helpful to set review flags by going to https://bugzilla.mozilla.org/attachment.cgi?id=756597&action=edit and setting the review drop down to ? and the reviewer field to :Margaret
https://developer.mozilla.org/en-US/docs/Developer_Guide/How_to_Submit_a_Patch#Getting_the_patch_reviewed
Assignee | ||
Updated•12 years ago
|
Attachment #756597 -
Flags: review?(margaret.leibovic)
Assignee | ||
Comment 5•12 years ago
|
||
Done. Thanks for the advice Kevin.
Comment 6•12 years ago
|
||
Thanks for the patch, Marcos! I'm going to wait until bug 877203 is fixed before reviewing it, but it looks like it is correct.
Let me know if you need help finding other bugs to work on!
Assignee | ||
Comment 7•12 years ago
|
||
Thank you Margaret for the help provided thus far. I hope bug 877203 gets fixed soon.
If you know of any other bugs that need some work, please let me know.
Comment 8•12 years ago
|
||
Comment on attachment 756597 [details] [diff] [review]
Bug 877783 patch
Marcos, I'm so sorry that bug 877203 has been taking forever to land. I'm clearing review for now, but hopefully we can revisit this whenever bug 877203 is finished.
Attachment #756597 -
Flags: review?(margaret.leibovic)
Comment 9•11 years ago
|
||
Reminding myself to look into landing this.
Flags: needinfo?(margaret.leibovic)
Reporter | ||
Comment 10•11 years ago
|
||
Er... Not sure if this landed somewhere else, but our Reader titles are, in fact, already bold :)
Comment 11•11 years ago
|
||
Doh, this was fixed by bug 930566. And I was also involved in that bug.
Sorry for the churn, Marcos. Hopefully you still learned something working on this bug. Let us know if you want help finding more bugs to work on!
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(margaret.leibovic)
Resolution: --- → DUPLICATE
Assignee | ||
Comment 12•11 years ago
|
||
Sure Margaret. Let me know if you need help with some other bugs.
Comment 13•11 years ago
|
||
(In reply to Marcos A. Di Pietro from comment #12)
> Sure Margaret. Let me know if you need help with some other bugs.
If you are interested, I just filed bug 999069, which is very similar to this bug. I was going to fix it myself, but I would love help :)
You can also take a look at this site for a big list of other mentor bugs:
http://www.joshmatthews.net/bugsahoy/?mobile=1
Thanks!
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•