615239 - Add "Get Help" SUMO link to Start Page

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, enhancement, P1)

Description

[Madhava Enros [:madhava]]

We should try to make SUMO content easier to access by putting a link on the Start Page, especially now that SUMO will be increasingly mobile friendly.

A possible presentation of it:
http://www.flickr.com/photos/madhava_work/5204410623/

Comment 1

[Caitlin Looney]

What's the status here? Who can implement this on the Start page?

Comment 2

[Aakash Desai [:aakashd]]

Adding some triage love.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

This patch adds "Get Help" and "Give Feedback" (from bug 615645)

Bug 615645 still needs the device and manufacturer details loaded into the new page.

Comment 4

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 507610 [details] [diff] [review]
patch

>diff --git a/chrome/content/aboutHome.xhtml b/chrome/content/aboutHome.xhtml
>     function init() {
>+      setupLinks();
>       initTabs();
>       initWeave();
>       initAddons();
>     }

I don't think this is a big deal but I wonder if it is possible to remove setupLinks from the init path, to have it on demand when the user clicks

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #4)
> Comment on attachment 507610 [details] [diff] [review]
> patch
> 
> >diff --git a/chrome/content/aboutHome.xhtml b/chrome/content/aboutHome.xhtml
> >     function init() {
> >+      setupLinks();
> >       initTabs();
> >       initWeave();
> >       initAddons();
> >     }
> 
> I don't think this is a big deal but I wonder if it is possible to remove
> setupLinks from the init path, to have it on demand when the user clicks

That's possible. I'm hoping the URLFormatter won't take much time.

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/36d63df6b32c

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: followup patch

Followup removes setupLinks and formats URL on demand

Comment 8

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed followup:
http://hg.mozilla.org/mobile-browser/rev/2f4962a5a048

Comment 9

[Aakash Desai [:aakashd]]

litmus testcase updated to regression test this bug:
 https://litmus.mozilla.org/show_test.cgi?id=12176