Closed Bug 731546 Opened 8 years ago Closed 8 years ago

Add link to marketplace on about:home

Categories

(Firefox :: General, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 14
Tracking Status
firefox13 + verified
firefox14 --- verified

People

(Reporter: Felipe, Assigned: fryn)

References

Details

(Whiteboard: [about-home][qa!])

Attachments

(1 file, 2 obsolete files)

The new about:home should have a button to open the apps marketplace; this will be the main entry point for Web Apps discovery in the product
Component: Theme → General
QA Contact: theme → general
Depends on: 711157
Whiteboard: [about-home]
No longer depends on: 711157
Assignee: nobody → fryn
Status: NEW → ASSIGNED
Attached image screenshot of patch w/ placeholders (obsolete) —
Comment on attachment 602259 [details]
screenshot of patch w/ placeholders

I absolutely have to give a positive feedback on the placeholder :)
Attachment #602259 - Flags: feedback+
Status: ASSIGNED → NEW
Assignee: fryn → nobody
Component: General → Web Apps
QA Contact: general → webapps
Assignee: nobody → fryn
Status: NEW → ASSIGNED
+<!ENTITY abouthome.appsButton.label      "Martellplace">

MartellPlace??
(In reply to Francesco Lodolo [:flod] from comment #4)
> +<!ENTITY abouthome.appsButton.label      "Martellplace">
> 
> MartellPlace??

Never mind, I was checking the strings and didn't see the screenshot ;-)
Component: Web Apps → General
QA Contact: webapps → general
Dolske asked me to write this patch and hopefully have it landed today or tomorrow.

The new ordering of the icons is ui-approved by limi and shorlander.

The requirement from product is to be able to turn on the marketplace icon using a pref. The patch checks for the hidden pref "browser.aboutHome.apps".
This pref is not for customization purposes. It simply exists by the apps team's request in the likely event that the marketplace launch happens between Firefox version launches. The pref check and hidden attribute will be removed once the marketplace launches.

The URL has been finalized as https://marketplace.mozilla.org/.

The marketplace icon is still a placeholder, as we have not been provided a final icon or silhouette from which to produce a matching icon for about:home. I will file a followup shortly for the final icon.
Attachment #602258 - Attachment is obsolete: true
Attachment #602259 - Attachment is obsolete: true
Attachment #608697 - Flags: review?(mak77)
Depends on: 738642
Comment on attachment 608697 [details] [diff] [review]
patch w/ placeholder icon

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

(In reply to Frank Yan (:fryn) from comment #6)
> The pref check and hidden attribute will
> be removed once the marketplace launches.

The patch looks good.

My doubt regarding this, is about branding.  Is the Marketplace something to be considered Firefox only? I wonder if for foreign branding we may have to always keep the button hidden option.
If so I'd probably consider inverting the option, so usually show the button unless a disable option is true and have other brands set the pref, than us.

::: browser/base/content/browser.js
@@ +2707,5 @@
>   */
>  function BrowserOnAboutPageLoad(document) {
>    if (/^about:home$/i.test(document.documentURI)) {
> +    // XXX when Marketplace is launched, remove this statement and the hidden
> +    // attribute set on the apps button in aboutHome.xhtml

I'd prefer if we'd have a bug tracking this, and the bug number be reported here instead of the n-th XXX comment that nobody will ever fix.
(In reply to Marco Bonardo [:mak] from comment #7)
> My doubt regarding this, is about branding.  Is the Marketplace something to
> be considered Firefox only? I wonder if for foreign branding we may have to
> always keep the button hidden option.

What exactly is your concern here? I don't see a meaningful dependency between official branding and the web apps marketplace.
Depends on: 738646
(In reply to Marco Bonardo [:mak] from comment #7)
> > +    // XXX when Marketplace is launched, remove this statement and the hidden
> > +    // attribute set on the apps button in aboutHome.xhtml
> 
> I'd prefer if we'd have a bug tracking this, and the bug number be reported
> here instead of the n-th XXX comment that nobody will ever fix.

I just filed bug 738646 and will add that to the comment when landing this.
I won't forget about it as I strongly dislike unnecessary prefs.

(In reply to Dão Gottwald [:dao] from comment #8)
> (In reply to Marco Bonardo [:mak] from comment #7)
> > My doubt regarding this, is about branding.  Is the Marketplace something to
> > be considered Firefox only? I wonder if for foreign branding we may have to
> > always keep the button hidden option.
> 
> What exactly is your concern here? I don't see a meaningful dependency
> between official branding and the web apps marketplace.

Marco and I just discussed his question on IRC. I don't think Marco has any major concern about this anymore. Feel free to read the #fx-team scrollback for more details.
Comment on attachment 608697 [details] [diff] [review]
patch w/ placeholder icon

ok, let's say we don't care till someone brings more meaningful reasoning about that.
Attachment #608697 - Flags: review?(mak77) → review+
Pushed to fx-team with bug number for removing temporary pref in comment.
https://hg.mozilla.org/integration/fx-team/rev/5ff4f574298e

Thank you for the quick review, Marco! :)
Whiteboard: [about-home] → [about-home][fixed-in-fx-team]
Comment on attachment 608697 [details] [diff] [review]
patch w/ placeholder icon

[Approval Request Comment]
User impact if declined: no marketplace link.
Risk to taking this patch: none.
String changes made by this patch: none.
Attachment #608697 - Flags: approval-mozilla-aurora?
Attachment #608697 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [about-home][fixed-in-fx-team] → [about-home]
To clarify - To test this, I need to enable browser.aboutHome.apps to true and check the about:home has a marketplace button with the appropriate icon. Then clicking that button should take me to marketplace.mozilla.org. Is this correct?
Actually, figured it out by enabling that pref. Looks like about:home does show the marketplace link in the bottom toolbar. The button does go to marketplace.mozilla.org. However, there's two outstanding issues for this bug:

- The icon shown is a giant X - Not the marketplace icon
- The code for this is enabled on FF 13, but should instead be enabled on FF 14 or later only
I don't think there's a problem with this code being enabled in Firefox 13 (I think that's the goal, actually) if the pref is off by default as expected. We will use the add-on hotfix to enable the button and add the appropriate image unless we can get that image sooner.
Whiteboard: [about-home] → [about-home][qa+]
No longer blocks: 731054
Verified on Nightly.
Status: RESOLVED → VERIFIED
Marketplace button is displayed in about:home when browser.aboutHome.apps pref is set on true.
Verified fixed on FF 13b3:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (X11; Linux i686; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:13.0) Gecko/20100101 Firefox/13.0
Whiteboard: [about-home][qa+] → [about-home][qa!]
You need to log in before you can comment on or make changes to this bug.