Closed Bug 1031018 Opened 8 years ago Closed 7 years ago

Add Firefox Marketplace to tools menu

Categories

(Firefox :: Menus, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 35
Iteration:
35.3

People

(Reporter: clouserw, Assigned: Gavin)

References

Details

User Story

User able to access highlighted Firefox Marketplace apps by going to the Tools menu and selecting "Apps"

Attachments

(1 file)

On Fennec you can click Tools -> Apps to load an intermediary page about apps which points to marketplace.firefox.com.  We should do the same thing in the desktop browser.  Since we don't have an intermediary page currently, we could just load marketplace.firefox.com.
Blocks: 1048519
No longer blocks: desktop-marketplace
User Story: (updated)
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Flags: qe-verify?
Flags: firefox-backlog+
(In reply to Wil Clouser [:clouserw] from comment #1)
> The URL for this is https://marketplace.firefox.com/discovery/

I assume this page does appropriate locale detection automatically?
Flags: needinfo?(clouserw)
yes
Flags: needinfo?(clouserw)
Flags: qe-verify? → qe-verify+
Attached patch patchSplinter Review
Screenshot: https://cloudup.com/ciQfOr2C1tm
Attachment #8492422 - Flags: ui-review?(philipp)
Attachment #8492422 - Flags: review?(MattN+bmo)
Comment on attachment 8492422 [details] [diff] [review]
patch

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

FWIW, the short "Apps" string look weird to me but it could just be because I'm used to certain strings in that menu. I understand this matches Fennec so that's fine but if I had to pick another name I would choose "App Marketplace".
Attachment #8492422 - Flags: review?(MattN+bmo) → review+
(In reply to Matthew N. [:MattN] from comment #5)
> Comment on attachment 8492422 [details] [diff] [review]
> patch
> 
> Review of attachment 8492422 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> FWIW, the short "Apps" string look weird to me but it could just be because
> I'm used to certain strings in that menu.

This seems particularly confusing because (1) all other "tools" are local ones, things that come built-in or that you've installed rather than gateways for discovering new stuff and (2) once you've installed apps from the marketplace, it makes even more sense to expect them to show up next to the other installed "tools".
The UI itself looks fine, but I'm not sure if that's the right solution to the problem.

Do we want to promote our marketplace? Then the tools menu is not a very effective choice since it is only visible by default on Mac and not very highly used in general.

Also, if we want to promote apps from the marketplace in the browser, we need to do a better job at explaining what will happen when you install an app and how you can access it afterwards.
Comment on attachment 8492422 [details] [diff] [review]
patch

Preliminary ui-r- until we figure out the issues raised in comment 7.
Attachment #8492422 - Flags: ui-review?(philipp) → ui-review-
(In reply to Philipp Sackl [:phlsa] from comment #7)
> Do we want to promote our marketplace?

Yes.

> Then the tools menu is not a very
> effective choice since it is only visible by default on Mac and not very
> highly used in general.

I agree, but it's not all we're doing. We're trying to be consistent with Add-ons in the general sense, so we're also going to fix bug 1031019.

> Also, if we want to promote apps from the marketplace in the browser, we
> need to do a better job at explaining what will happen when you install an
> app and how you can access it afterwards.

This is reasonable, but shouldn't the Marketplace itself do this?
Let's definitely do a great job explaining what happens when you install an app.
MDN has made a good start: https://developer.mozilla.org/en-US/Marketplace/Options/Open_web_apps_for_desktop 

Where should some consumer info be placed?

FYI - Desktop apps already install, and are managed in the same way apps are managed on a desktop machine.
See this video for questions: https://www.youtube.com/watch?v=dO-l5odB4hM
Comment on attachment 8492422 [details] [diff] [review]
patch

Asking for reconsideration given comment 9
Attachment #8492422 - Flags: ui-review- → ui-review?(philipp)
I went ahead and pushed this as-is. Philipp, please do let me know if you want me to follow-up and adjust this at all.

https://hg.mozilla.org/integration/fx-team/rev/39ee3eba02cf
Target Milestone: --- → Firefox 35
Comment on attachment 8492422 [details] [diff] [review]
patch

After-the-fact-ui-r+ :)

I'll file some bugs for the marketplace team to make it a little more desktop-friendly.
Attachment #8492422 - Flags: ui-review?(philipp) → ui-review+
https://hg.mozilla.org/mozilla-central/rev/39ee3eba02cf
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Iteration: --- → 35.3
QA Contact: bogdan.maris
I did some exploratory testing around this implementation and did not found any issues. I tested using latest Nightly on Windows 7 64bit, Ubuntu 14.04 32bit and Mac OS X 10.9.5. 
Marking this as verified based on my testing.
Status: RESOLVED → VERIFIED
Release Note Request (optional, but appreciated)
[Why is this notable]: With bug 1031019, the Firefox Marketplace is promoted in desktop Firefox
[Suggested wording]: Access the Firefox Marketplace from the Tools menu and optional toolbar button.
[Links (documentation, blog post, etc)]: https://marketplace.firefox.com/discovery/
relnote-firefox: --- → ?
noted from bug 1031019, no need to do it twice.
relnote-firefox: ? → ---
You need to log in before you can comment on or make changes to this bug.