Add link to marketplace on about:home

VERIFIED FIXED in Firefox 13

Status

()

Firefox
General
VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: Felipe, Assigned: fryn)

Tracking

Trunk
Firefox 14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox13+ verified, firefox14 verified)

Details

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

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

5 years ago
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

Updated

5 years ago
Component: Theme → General
QA Contact: theme → general

Updated

5 years ago
Depends on: 711157
Whiteboard: [about-home]

Updated

5 years ago
No longer depends on: 711157

Updated

5 years ago
Depends on: 711157
(Assignee)

Updated

5 years ago
Assignee: nobody → fryn
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Created attachment 602258 [details] [diff] [review]
patch w/ placeholders; waiting for official icons & branding
(Assignee)

Comment 2

5 years ago
Created attachment 602259 [details]
screenshot of patch w/ placeholders
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

Updated

5 years ago
Assignee: fryn → nobody
Component: General → Web Apps
QA Contact: general → webapps

Updated

5 years ago
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 ;-)

Updated

5 years ago
Component: Web Apps → General
QA Contact: webapps → general
(Assignee)

Comment 6

5 years ago
Created attachment 608697 [details] [diff] [review]
patch w/ placeholder icon

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)
(Assignee)

Updated

5 years ago
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.
(Assignee)

Updated

5 years ago
Depends on: 738646
(Assignee)

Comment 9

5 years ago
(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+
(Assignee)

Comment 11

5 years ago
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]
(Assignee)

Comment 12

5 years ago
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+
https://hg.mozilla.org/mozilla-central/rev/5ff4f574298e
https://hg.mozilla.org/releases/mozilla-aurora/rev/7ebf7cd335f2
status-firefox13: --- → fixed
tracking-firefox13: --- → +
Target Milestone: --- → Firefox 14
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [about-home][fixed-in-fx-team] → [about-home]

Updated

5 years ago
Blocks: 614106
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

Comment 16

5 years ago
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+]

Updated

5 years ago
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
status-firefox13: fixed → verified
Whiteboard: [about-home][qa+] → [about-home][qa!]

Updated

5 years ago
status-firefox14: --- → verified
You need to log in before you can comment on or make changes to this bug.