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)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 930566

People

(Reporter: ibarlow, Assigned: marcosadp)

References

Details

(Whiteboard: [mentor=margaret][lang=css])

Attachments

(1 file)

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
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
Depends on: 877203
OS: Mac OS X → Android
Hardware: x86 → ARM
Whiteboard: [mentor=margaret][lang=css]
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.
Assignee: nobody → marcosadp
Attached patch Bug 877783 patchSplinter Review
Changes line 66 of /mobile/android/themes/core/aboutReader.css from font-weight: 300; to font-weight: bold;
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
Attachment #756597 - Flags: review?(margaret.leibovic)
Done. Thanks for the advice Kevin.
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!
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 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)
Reminding myself to look into landing this.
Flags: needinfo?(margaret.leibovic)
Er... Not sure if this landed somewhere else, but our Reader titles are, in fact, already bold :)
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
Sure Margaret. Let me know if you need help with some other bugs.
(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!
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: