530850 - Add link from mo.sumo to desktop support

Status: VERIFIED FIXED

(support.mozilla.org :: Mobile, task)

Description

[James Socol [:jsocol, :james]]

(From bug 517627) Need to add a link to the mo.sumo themes back to desktop support.

Comment 1

[Alex Buchanan [:abuchanan]]

CC'ing Caitlin and David to work out any design decisions:

An example is the Thunderbird button here: http://www.flickr.com/photos/djst/4098030838/

Afaik, ff-mobile doesn't have it's own logo, so the usual FF logo can be used.

Caitlin, want to comment on copy for that block?

Anything else needed?

Comment 2

[Chris Ilias [:cilias]]

Should this be part of the template or as a module (like have with Thunderbird)? There is already bug 529730 [Create module for mobile].

Comment 3

[David Tenser [:djst]]

I'm mostly indifferent, but a module worked before so seems fine for this too. 

What we need is the graphics -- it could look just like the Thunderbird button but with a Firefox logo. Would be interested to hear if garrety/jslater has any other comments.

Comment 4

[David Tenser [:djst]]

Some clarifications might be needed here: when I say "Thunderbird button", I mean the area linking to Thunderbird, including the Thunderbird logo and the "diagonal, gray background" box. Something like that but for Firefox would work well. 

Ideally, we'd have some symbols to indicate "mobile" vs "desktop" in which case this box would use the "desktop" symbol. We could then link back to the mobile site from desktop support (support.mozilla.com) using the "mobile" symbol. That's probably out of scope for this bug though, but a thought I wanted to document here nevertheless...

Comment 5

[Alex Buchanan [:abuchanan]]

(In reply to comment #1)
> Afaik, ff-mobile doesn't have it's own logo, so the usual FF logo can be used.

Oops, I forgot about this,
http://creative.mozilla.org/designs/346

A smaller version can be found here,
https://www-trunk.stage.mozilla.com/en-US/mobile/

Comment 6

[Paul Craciunoiu [:paulc]]

(In reply to comment #5)
> Oops, I forgot about this,
> http://creative.mozilla.org/designs/346
> 
> A smaller version can be found here,
> https://www-trunk.stage.mozilla.com/en-US/mobile/
Aw, it's so cute! Sorry for the spam but I couldn't help it :)

Comment 7

[John Slater]

(In reply to comment #6)
> Aw, it's so cute! Sorry for the spam but I couldn't help it :)

Thanks Paul! Despite the Pocketfox's cuteness, it's not really intended to be a distinct logo for the mobile Firefox. The line may be a little blurry, but the idea is to use Pocketfox more in a promotional context...as far as the true logo goes, it's the official Firefox one just like always.

If we go with the true logo, there are plenty of places around mozilla.com that use a format very much like what David is describing in comment #4 (http://www.mozilla.com/en-US/firefox/security/, for example). Seems like it'd be pretty easy to just grab the code from one of those pages and repurpose it here (with new text, and without a green download button of course).

For copy, we'd need a headline and detail text to fit the usual format. Here's a rough pass...feel free to edit as needed-

Headline: Firefox Desktop Support
Detail: For answers to your questions about the desktop version of Firefox, visit our _desktop Firefox help page_.

Comment 8

[David Tenser [:djst]]

Alternative text:

Headline: Firefox Desktop Support
Detail: Looking for help with Firefox on your computer? Visit our _desktop Firefox help site_.

Paul: this should definitely be implemented as a module so we can tweak the wording after we push.

Comment 9

[John Slater]

The alternative text in comment #8 is much better - thanks David!

Comment 10

[Caitlin Looney]

I'd like to be clear on who is owning this. Alex, are you on this? Thanks for letting me know.

Comment 11

[Alex Buchanan [:abuchanan]]

(In reply to comment #10)
> I'd like to be clear on who is owning this. Alex, are you on this? Thanks for
> letting me know.

Not me, I've never touched the SUMO codebase, although I can try if needed.

Comment 12

[David Tenser [:djst]]

What we need is the graphics, i.e. http://svn.mozilla.org/projects/sumo/trunk/webroot/styles/mozms/sidebar-tb.png but with a Firefox logo (if no one has a better suggestion for how to point people to the desktop version of Firefox, that is!).

Once we have that logo, James or Paul C can take this bug.

Comment 13

[John Slater]

(In reply to comment #12)
> What we need is the graphics, i.e.
> http://svn.mozilla.org/projects/sumo/trunk/webroot/styles/mozms/sidebar-tb.png
> but with a Firefox logo (if no one has a better suggestion for how to point
> people to the desktop version of Firefox, that is!).
> 
> Once we have that logo, James or Paul C can take this bug.

Steven, can you do this? I think it's a pretty quick fix, then James or Paul can take it from there.

Thanks!

Comment 14

[Steven Garrity [:sgarrity]]

David, can you use this image directly? http://www.mozilla.com/img/tignish/firefox/download-background.png It's a touch wider than the other sidebar backgrounds, but it may work as is. If you need an updated version for the narrower column, let me know.

Comment 15

[David Tenser [:djst]]

James, could you test the above image and see if it works?

Comment 16

[James Socol [:jsocol, :james]]

Attachment: css patch

You'll need to add the image from comment 14 (and I'll attach) at the location specified in the patch. You'll also need to create a module in tiki-admin_modules.php. I've created the module on mobile.support-stage.m.o, with these values:

Name: desktophelp
Title: Firefox Desktop Help
Content <<<STOP
	<div id="desktophelp-title" class="title">{tr}Firefox Desktop Support{/tr}</div>
	<div id="desktophelp-content">{tr}Looking for help with Firefox on your computer? Visit our <a href="http://support.mozilla.com/">desktop
Firefox help site</a>.{/tr}</div>
STOP

Then you'll need to assign it with these parameters:
Module: desktophelp
Order: 1
Rows: 10
Params: theme=mozms.css

Comment 17

[James Socol [:jsocol, :james]]

Attachment: module background

Place at
webroot/styles/mozms/sidebar-desktop.png

Comment 18

[Paul Craciunoiu [:paulc]]

Comment on attachment 416605 [details] [diff] [review]
css patch

Looks good on Ubuntu 9.10 (Fx3.5), WinXP (Fx3.5, IE7,8, Chrome, Opera).
On IE6 the right side of the image gets clipped, but that would have to be fixed in a separate stylesheet.

Comment 19

[James Socol [:jsocol, :james]]

r57824.

Comment 20

[Stephen Donner [:stephend] Not actively reading bugmail]

Does this need to be assigned before QA can verify?

Comment 21

[James Socol [:jsocol, :james]]

Assigned. Sorry 'bout that Stephen.

Comment 22

[James Socol [:jsocol, :james]]

But I think it's being caught in the cache while logged out.

Comment 23

[Stephen Donner [:stephend] Not actively reading bugmail]

I see it while logged in, too; will wait until it clears through the other caches to verify (unless we need to push ASAP?)

Comment 24

[James Socol [:jsocol, :james]]

If we don't see it by tomorrow we'll ask IT to flush the caches.

Comment 25

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Screenshot of IE 6 -- clipping issue

Comment 26

[Stephen Donner [:stephend] Not actively reading bugmail]

Sigh ^^^

Comment 27

[Paul Craciunoiu [:paulc]]

Attachment: ie6 fix

This should do it.

Comment 28

[James Socol [:jsocol, :james]]

Comment on attachment 417550 [details] [diff] [review]
ie6 fix

This does not work for me, even when I modify the selector to match the specificity of others in the style sheet and add background-color:#000; to check that it's working.

Comment 29

[James Socol [:jsocol, :james]]

Attachment: change padding in ie6

This changes the padding to be friendlier with IE6's broken box model, and matches the selector specificity of other rules in this style sheet.

Comment 30

[Paul Craciunoiu [:paulc]]

Comment on attachment 417798 [details] [diff] [review]
change padding in ie6

Woo it works on my VM too!

Comment 31

[James Socol [:jsocol, :james]]

r57916

Comment 32

[Vishal Kamdar [:vish_moz]]

Verified, FIXED.
IE 6 logo clipping issue has been fixed.