877783 - Switch Reader Mode title to bold for Clear Sans

Status: RESOLVED DUPLICATE of bug 930566

Description

[Ian Barlow (:ibarlow)]

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

[:Margaret Leibovic]

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

Comment 2

[Marcos A. Di Pietro]

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.

Comment 3

[Marcos A. Di Pietro]

Created attachment 756597 [details] [diff] [review]
Bug 877783 patch

Changes line 66 of /mobile/android/themes/core/aboutReader.css from font-weight: 300; to font-weight: bold;

Comment 4

[Kevin Brosnan [:kbrosnan]]

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

Comment 5

[Marcos A. Di Pietro]

Done. Thanks for the advice Kevin.

Comment 6

[:Margaret Leibovic]

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!

Comment 7

[Marcos A. Di Pietro]

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

[:Margaret Leibovic]

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.

Comment 9

[:Margaret Leibovic]

Reminding myself to look into landing this.

Comment 10

[Ian Barlow (:ibarlow)]

Er... Not sure if this landed somewhere else, but our Reader titles are, in fact, already bold :)

Comment 11

[:Margaret Leibovic]

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!

Comment 12

[Marcos A. Di Pietro]

Sure Margaret. Let me know if you need help with some other bugs.

Comment 13

[:Margaret Leibovic]

(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!