414.08 KB, image/png
240.04 KB, image/png
242.13 KB, image/png
656 bytes, patch
|Details | Diff | Splinter Review|
Created attachment 332902 [details] Screenshot This was mentioned over in https://bugzilla.mozilla.org/show_bug.cgi?id=442182#c26, and is a regression from that bug. Steps to Reproduce: 1. Load http://support-stage.mozilla.org/en-US/kb/Other+Firefox+support 2. Look at the white "Other Firefox Support" link against the background See screenshot.
FWIW, in order to disable commments on that article, I've switched it to mozad.css.
As per Comment #1, it'll be fine on production. However, this doesn't seem to go well with the "How to Contribute" category that the main title has. Regardless, it's ok for now, downgrading to normal and assigning to Laura since I imagine I'll be gone before we come up with a good solution (I'm sure djst will think of something). Easiest thing to do is to shift the header down 15 pixels so that the blue is lower. Not the best solution, but the easiest.
Assignee: bkrausz → laura
Severity: major → normal
Created attachment 332971 [details] when logged out The same problem still appears when logged out. See screenshot.
Not sure I understand why this is on a different template than the other pages - the gray header image also looks kind of funny on the blue.
Target Milestone: 0.6.2 → 0.6.3
Created attachment 335535 [details] [diff] [review] Tweaks to recolor links and move background on "mini" style variant It looks like some pages have a "mini" style variant, where the search bar is hidden and there's no large title at the top - not sure why it's this way. Also, I wasn't able to reproduce Chris Ilias' screenshot differences between logged in and logged out. As a quick fix, I tweaked the CSS in this patch to give the sidebar links a blue color like other links on the page when the "mini" variant is in effect. Additionally, I pushed the background up a bit in order to make the grey section rule look better.
Attachment #335535 - Flags: review?(laura)
Comment on attachment 335535 [details] [diff] [review] Tweaks to recolor links and move background on "mini" style variant WFM
Attachment #335535 - Flags: review?(laura) → review+
Have people been changing templates? http://support.mozilla.com/en-US/kb/Other+Firefox+support and staging are both fine, seemingly without this fix...
Hmm.. might need to look into that a little more - seems confusing, since that page is "Other Firefox support" but the big title says "How to Contribute", which is the next link down in the sidenav.
see comment 1
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
I don't know how to verify this until the change to use the mozad.css stylesheet in comment 1 is reverted.
Huh, I must not understand comment 1, because I'm not seeing mozad.css anywhere on the page on trunk
Okay, I've switched it back on prod.
(In reply to comment #14) > Okay, I've switched it back on prod. Chris, could you do this on stage, too, so that I can verify it? Thanks! http://support-stage.mozilla.org/en-US/kb/Other+Firefox+support
Looks like it's already applied to staging.
Verified FIXED; don't know what I was thinking/seeing. Thanks, Chris.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.