Closed
Bug 535330
Opened 15 years ago
Closed 15 years ago
Update site footers to match mozilla.com
Categories
(support.mozilla.org :: Knowledge Base Software, task, P3)
support.mozilla.org
Knowledge Base Software
Tracking
(Not tracked)
VERIFIED
FIXED
1.5.1
People
(Reporter: jsocol, Assigned: Tobbi)
References
()
Details
(Whiteboard: [good first bug])
Attachments
(3 files, 4 obsolete files)
2.63 KB,
patch
|
Details | Diff | Splinter Review | |
44.02 KB,
image/png
|
Details | |
1.29 KB,
patch
|
Details | Diff | Splinter Review |
Mozilla.com has a new "Mobile" section in the footer, and we should update our templates to match.
(NB: we should keep the locale-specific link to mobile.support.m.c.)
Assignee | ||
Comment 1•15 years ago
|
||
I've made changes to the footer links to fit the mozilla.com homepage
Assignee: nobody → tobbi.bugs
Status: NEW → ASSIGNED
Attachment #420306 -
Flags: review?(paulc)
Attachment #420306 -
Flags: review?(james)
Comment 2•15 years ago
|
||
The "final" header/footer--i.e. the one we're shipping with come 3.6's launch, can be seen on https://www-3.6.stage.mozilla.com/en-US/, so it'd be best to match that.
Assignee | ||
Comment 3•15 years ago
|
||
Matching the footer links against the Firefox 3.6 staging site
Attachment #420306 -
Attachment is obsolete: true
Attachment #420370 -
Flags: review?(paulc)
Attachment #420370 -
Flags: review?(james)
Attachment #420306 -
Flags: review?(paulc)
Attachment #420306 -
Flags: review?(james)
Comment 4•15 years ago
|
||
Comment on attachment 420370 [details] [diff] [review]
Followup patch matching against 3.6 staging of mozilla.com
r59242
WFM by comparison with https://www-3.6.stage.mozilla.com/en-US/
Tobbi: when you file patches, please cd to "trunk" and run "svn diff" from there.
Attachment #420370 -
Attachment is obsolete: true
Attachment #420370 -
Flags: review?(paulc)
Attachment #420370 -
Flags: review?(james)
Comment 5•15 years ago
|
||
Here's the same patch from root of trunk.
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Target Milestone: --- → 1.5.1
Comment 6•15 years ago
|
||
Comment 7•15 years ago
|
||
Verified FIXED on http://support-stage.mozilla.org/en-US/kb/.
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 8•15 years ago
|
||
Paul, would you mind getting this onto the fennec branch, as well?
Comment 9•15 years ago
|
||
r59391 (fennec). Also changed resolution so QA can re-verify on the mobile site.
Status: VERIFIED → RESOLVED
Closed: 15 years ago → 15 years ago
And also verified on http://mobile.support-stage.mozilla.org/en-US/kb/.
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 11•15 years ago
|
||
I found out that one of the links was wrong, fixing it with this patch.
Attachment #420966 -
Flags: review?(paulc)
Attachment #420966 -
Flags: review?(james)
Assignee | ||
Updated•15 years ago
|
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Comment 12•15 years ago
|
||
r59502
Thanks for fixing that Tobbi.
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Attachment #420966 -
Flags: review?(paulc)
Attachment #420966 -
Flags: review?(james)
Attachment #420966 -
Flags: review+
:-(
I just realized that http://www.mozilla.com/firefox/underthehood/ is a 3.6-only page, so it'll be a 404 until Mozilla.com 3.6 ships and we've merged with trunk.
Sorry, reopening for comment 13; I should've noticed that earlier. Let's just remove that link for now until after 3.6 ships.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 15•15 years ago
|
||
I removed the Underthehood link at first, because it's not on staging yet.
Attachment #421403 -
Flags: review?(paulc)
Attachment #421403 -
Flags: review?(james)
Updated•15 years ago
|
Attachment #421403 -
Flags: review?(paulc)
Attachment #421403 -
Flags: review?(james)
Attachment #421403 -
Flags: review+
Comment 16•15 years ago
|
||
Comment on attachment 421403 [details] [diff] [review]
Removing the underthehood link
Thanks Tobbi. r59722
Hope it's all good now.
Updated•15 years ago
|
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Tobbi: mobile still needs the fix, though :-)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 18•15 years ago
|
||
I'm confused.
Reporter | ||
Comment 19•15 years ago
|
||
Paul: Can you check Tobbi's patch into the fennec branch? (Or possibly both patches, combined? I'm not sure what ended up there already).
Comment 20•15 years ago
|
||
Yes, two more patches in a row on trunk (marking them obsolete). Filed as a patch and committed on fennec.
r59926
Attachment #420966 -
Attachment is obsolete: true
Attachment #421403 -
Attachment is obsolete: true
Updated•15 years ago
|
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Verified FIXED on http://mobile.support-stage.mozilla.org/en-US/kb/ and again on https://support-stage.mozilla.org/en-US/kb/.
Status: RESOLVED → VERIFIED
Reporter | ||
Updated•15 years ago
|
Status: VERIFIED → RESOLVED
Closed: 15 years ago → 15 years ago
Reporter | ||
Comment 22•15 years ago
|
||
Reopening to commit to branches/1.5.1_temp. Will re-resolve after patches are committed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Comment 23•15 years ago
|
||
On branch, r62437
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•