Closed Bug 1007431 Opened 6 years ago Closed 6 years ago

Add menu item that links to SUMO help

Categories

(Firefox for Android :: General, enhancement)

All
Android
enhancement
Not set

Tracking

()

VERIFIED FIXED
Firefox 32

People

(Reporter: liuche, Assigned: mfinkle)

References

Details

Attachments

(1 file)

Found bug 725006 that discussed this in the past. I assume that we want to put it in the menu?

I would be inclined to stick "Help" in the next to last position, right above Settings, because this keeps Settings easy to tap (and it's probably going to be more used than the Help menu).

The SUMO FAQ would also be more useful to link to than the SUMO main page - if they don't find what they're looking for, they can search there, instead of getting dropped into the main SUMO page and making them type/search from there.

We'll need to get the Mobile SUMO FAQ updated, though, because the current page claims it's not maintained.
Roland, is updating the Mobile SUMO FAQ something that you can look into?
Flags: needinfo?(rtanglao)
(In reply to Chenxia Liu [:liuche] from comment #1)
> Roland, is updating the Mobile SUMO FAQ something that you can look into?

hi Chenxia: most definitely, jsavage (who I have CC'd), our SUMO content editor,  will write it (or find the best way to modify an existing article which could even be the existing mobile faq) and i will review it!
Flags: needinfo?(rtanglao)
(In reply to Roland Tanglao :rolandtanglao from comment #3)

> :liuche this is for FF32 right?

Yes
Assignee: nobody → mark.finkle
Attached patch help-menu v0.1Splinter Review
The patch adds the menu item for all layouts. I put made it the last item in the menu, except when the Guest Browsing menu is active. This matches several other apps that I found using Help menus too.

The code opens the main SUMO page for Firefox for Android. The same one used by the bookmark and the about: page link to Support. This seems better suited for "Help" than the FAQ page.

Review to Newman mainly for the locale stuff I cribbed from some Sync code. Otherwise, it's a simple patch.
Attachment #8427227 - Flags: review?(rnewman)
Comment on attachment 8427227 [details] [diff] [review]
help-menu v0.1

Review of attachment 8427227 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/locales/en-US/android_strings.dtd
@@ +228,5 @@
>  <!ENTITY pref_update_autodownload_wifi "Only over Wi-Fi">
>  <!ENTITY pref_update_autodownload_never "Never">
>  <!ENTITY pref_update_autodownload_always "Always">
>  
> +<!ENTITY help "Help">

Might want to call this something more clearly localizable, like menu_help. Not every language has noun == imperative verb.
Attachment #8427227 - Flags: review?(rnewman) → review+
Severity: normal → enhancement
Status: NEW → ASSIGNED
Hardware: ARM → All
Switched to help_menu and added a localization hint
https://hg.mozilla.org/integration/fx-team/rev/480860e8a7f2
Comment on attachment 8427227 [details] [diff] [review]
help-menu v0.1

Review of attachment 8427227 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/BrowserApp.java
@@ +1247,2 @@
>                              menu.findItem(R.id.settings).setEnabled(true);
> +                            menu.findItem(R.id.help).setEnabled(true);

Drive-by: We could techincally still open a tab and get a page ready to load if gecko isn't running yet. But maybe it would annoy users that their help page doesn't immediately load?
(In reply to :Margaret Leibovic from comment #8)
> Comment on attachment 8427227 [details] [diff] [review]
> help-menu v0.1
> 
> Review of attachment 8427227 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: mobile/android/base/BrowserApp.java
> @@ +1247,2 @@
> >                              menu.findItem(R.id.settings).setEnabled(true);
> > +                            menu.findItem(R.id.help).setEnabled(true);
> 
> Drive-by: We could techincally still open a tab and get a page ready to load
> if gecko isn't running yet. But maybe it would annoy users that their help
> page doesn't immediately load?

I am willing to wait and see how things go. We could do the same for Settings too. We do know that some people dislike the disabled Settings menu.
https://hg.mozilla.org/mozilla-central/rev/480860e8a7f2
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 32
Verified as fixed in build Nightly (2014-05-26)
Device: Alcatel One Touch (Android 4.1.2)
Status: RESOLVED → VERIFIED
Blocks: 1021556
We've compiled a list of the most popular Firefox for Android articles as well as the YouTube videos here: https://support.mozilla.org/kb/popular-topics-firefox-android. The "Help" menu item is now pointing to this new page, but we can change it back if you think the product page was better.
You need to log in before you can comment on or make changes to this bug.