Closed Bug 1248853 Opened 7 years ago Closed 7 years ago

Provide options for users to view URL on TV Web Apps listings

Categories

(Firefox OS Graveyard :: Gaia::TV, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-b2g:2.5+, b2g-v2.5 verified, b2g-master verified)

VERIFIED FIXED
blocking-b2g 2.5+
Tracking Status
b2g-v2.5 --- verified
b2g-master --- verified

People

(Reporter: jcheng, Assigned: jyeh)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(3 files)

base on latest discussion with various groups, we should add an option for users to "view URL" on TV Web Apps listings
+1 for Tori's Option # 3 - "Place URL inside option menu"

I think this approach would be best, given the URLs we have today. For illustration, here is an example of two of our current content URLs:

Cliggo - http://music-dev.cliggo.com/includes/app_specials/tv/?device=mobile&tv=panasonic
Bust A Move - http://d3divlwnjap65x.cloudfront.net/
Tori,
blocking-b2g: --- → 2.5+
Whiteboard: [ft:conndevices]
Hi Tori,
Could you update the UX spec to reflect this so Joseph can work on that?
Thanks
Flags: needinfo?(tchen)
Please check the updated spec here (v2.3 p26~27)
https://drive.google.com/open?id=0B4dMhI4hp32OWllCWHBkQXlnbFE
Flags: needinfo?(tchen) → needinfo?(jocheng)
Blocks: 1250032
Comments from previous discussion in email thread for record

proposals
https://docs.google.com/a/mozilla.com/presentation/d/18GKw5fc6Z3tvqdSNerGPnYHe-ZaLNqBK6zNT5h0sftk/edit?usp=sharing

Joe: #3 looks good, like what we discussed in the morning

Michael: +1 for Option 3 - "Place URL in option menu"

Elvin:  For the interim, the URL should be 'visible' to users, but we should process that URL as we would in any other context. Longer term something like Option 1 or 2 would be preferred as it makes most clear who is responsible for the content and sets users expectations right off the bat.
No longer blocks: 1250032
Joseph, please checkout this spec and see if there is any missing piece. Question: for hosted app, do we know their launch URL in manifest from server side?
Assignee: nobody → jyeh
Flags: needinfo?(jyeh)
Flags: needinfo?(jocheng)
Blocks: 1250057
Yes, api will give us the url of the hosted app.

However, in the spec p26, I'm not sure if we can change the text of ok button in a native confirm dialog. Or should this be handled by the system app?
Flags: needinfo?(jyeh) → needinfo?(ehung)
After discussion with Luke and Ricky, it seems to be more appropriate to handled by the system app.
Flags: needinfo?(ehung)
Ricky,
Please help this, this is priority 1. Thanks
Flags: needinfo?(rchien)
Priority: -- → P1
After offline discussion, I will be a mentor for this issue helping mentee work on smart-system part.
Flags: needinfo?(rchien)
Hi Evelyn,
PLease help assign developer on this bug. Thanks
Flags: needinfo?(ehung)
Blocks: TV_P1
Let's take option 3.
@tori, please provide your UX spec here, so Joseph can start working on it. Thanks!
Flags: needinfo?(ehung) → needinfo?(tchen)
Here is the latest ux spec (V2.5, p26.27), thanks!
https://drive.google.com/open?id=0B4dMhI4hp32OWllCWHBkQXlnbFE
Flags: needinfo?(tchen)
Status: NEW → ASSIGNED
Comment on attachment 8726032 [details] [review]
[gaia] jostw:josyeh/Marketplace/master/Bug1248853 > mozilla-b2g:master

Hi Luke, this patch will add a context menu to show the link of the website. Please take a look at it, thanks!
Attachment #8726032 - Flags: review?(lchang)
Blocks: 1253145
Comment on attachment 8726032 [details] [review]
[gaia] jostw:josyeh/Marketplace/master/Bug1248853 > mozilla-b2g:master

Nice work!
Attachment #8726032 - Flags: review?(lchang) → review+
Landed in gaia master.

https://github.com/mozilla-b2g/gaia/commit/f41e046a5c6032d9ce281fdb96256be7fc9b0c0f
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment on attachment 8726032 [details] [review]
[gaia] jostw:josyeh/Marketplace/master/Bug1248853 > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):1248853
[User impact] if declined: cannot see url of the website
[Testing completed]: Yes
[Risk to taking this patch] (and alternatives if risky): Low
[String changes made]: Yes
Attachment #8726032 - Flags: approval-gaia-v2.5?(jocheng)
Comment on attachment 8726032 [details] [review]
[gaia] jostw:josyeh/Marketplace/master/Bug1248853 > mozilla-b2g:master

Approve for TV 2.5
Attachment #8726032 - Flags: approval-gaia-v2.5?(jocheng) → approval-gaia-v2.5+
You need to log in before you can comment on or make changes to this bug.