Defect - Give site authors the ability to link Windows store apps w/ their page for IE10 parity

RESOLVED FIXED in Firefox 27

Status

defect
P2
normal
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: bbondy, Assigned: kjozwiak)

Tracking

Trunk
Firefox 27
x86_64
Windows 8.1
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: feature=defect c=tbd u=tbd p=0)

Attachments

(3 attachments, 3 obsolete attachments)

IE10 can link HTML5 app w/ a page, "Get app for this site".
This would probably add a menu item to one of the appbar icons.

<meta name="msApplication-ID" content="App"/>
<meta name="msApplication-PackageFamilyName" content="ZeptoLabUKLimited.CutTheRope_sq9zxnwrk84pj"/>
Whiteboard: [preview-triage]
Posted image images.jpg
There is text needed for the menu "Get app for this site" so if we want this later it can't be uplifted past mozilla-central.  Attached a screenshot of how it looks in IE.
kjozwiak this is probably a good bug for you if you're interested. I think we'd just show/hide the menu item depending on if the site has meta tags in it.  See the attachment image in Comment 1. 

You'd check for these meta tags "msApplication-ID", and "msApplication-PackageFamilyName" which look like this in the head of the current page:

<meta name="msApplication-ID" content="microsoft.build.App" />
<meta name="msApplication-PackageFamilyName" content="microsoft.build_8wekyb3d8bbwe"/>

And if they exist, then you'd selectively show/hide the menu item in the appbar settings icon.

If the menu is clicked you'd open the link, something like this:
location.href = "ms-windows-store:PDP?PFN=microsoft.build_8wekyb3d8bbwe"
Sounds good, thanks Brian! Appreciate it.

I will start this one when I am finished with Bug 885383 which should be completed by this week.
Assignee: nobody → kamiljoz
Summary: Give site authors the ability to link Windows store apps w/ their page for IE10 parity → Defect - Give site authors the ability to link Windows store apps w/ their page for IE10 parity
Whiteboard: [preview-triage] → [preview-triage] feature=defect c=tbd u=tbd p=0
Blocks: metrov2defect&change
No longer blocks: metrov1backlog
We'd really really like a patch, but won't block on this. :)
Whiteboard: [preview-triage] feature=defect c=tbd u=tbd p=0 → feature=defect c=tbd u=tbd p=0
Posted file detecting meta tags (obsolete) —
work in progress: code to detect meta tags
Posted patch Current WIP (obsolete) — Splinter Review
Attached current WIP
Went through testing and everything seems like its working:
- Ensured that the menu item doesn't appear when the meta data isn't found (tried several popular websites)
- Ensured that the menu item appeared correctly when the appropriate meta data was found
- Ensured that clicking on the menu item took the user to the MS store and displayed the correct application
- Ensured that the other menu items still worked without any issues

You can use this page that I created for testing:
https://s3.amazonaws.com/coding/METAdisabled.html
Attachment #802086 - Attachment is obsolete: true
Attachment #808771 - Attachment is obsolete: true
Attachment #810336 - Flags: review?(jmathies)
Forgot to add the page that has META data, this will open the "Cut the Rope" game in the MS store:

https://s3.amazonaws.com/coding/METAenabled.html
Yuan, please let me know if you would like this changed. Used the same label as the original screenshot Brian added.
Attachment #810340 - Flags: ui-review?(ywang)
Comment on attachment 810336 [details] [diff] [review]
Checking meta data then adds appropriate menu entry that takes user to MS store

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

::: browser/metro/base/content/ContextCommands.js
@@ +265,5 @@
> +      Browser.selectedBrowser.contentWindow.document.location = storeLink;
> +    }
> +  },
> +
> +  getStoreLink: function cc_getStoreLink() {

you might want to add a comment header here explaining what this is about.

::: browser/metro/locales/en-US/chrome/browser.dtd
@@ +16,5 @@
>  <!ENTITY appbarErrorConsole.label   "Open error console">
>  <!ENTITY appbarJSShell.label        "Open JavaScript shell">
>  <!ENTITY appbarFindInPage2.label    "Find in page">
>  <!ENTITY appbarViewOnDesktop2.label "View on desktop">
> +<!ENTITY appbarMSMetaData2.label "Get app for this site">

nit - line up
Attachment #810336 - Flags: review?(jmathies) → review+
Jim, could you push to the try server?
Attachment #810336 - Attachment is obsolete: true
Attachment #811148 - Flags: review+
Attachment #810340 - Flags: ui-review?(ywang) → ui-review+
fx-team
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/ae03c0e61c14
Keywords: checkin-needed
Whiteboard: feature=defect c=tbd u=tbd p=0 → feature=defect c=tbd u=tbd p=0[fixed-in-fx-team]
Blocks: metrov1it16
No longer blocks: metrov2defect&change
Status: NEW → ASSIGNED
Priority: -- → P2
QA Contact: jbecerra
https://hg.mozilla.org/mozilla-central/rev/ae03c0e61c14
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: feature=defect c=tbd u=tbd p=0[fixed-in-fx-team] → feature=defect c=tbd u=tbd p=0
Target Milestone: --- → Firefox 27
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.