Closed Bug 1235135 Opened 7 years ago Closed 7 years ago

[TV][2.5] Hide price text in preview area

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(feature-b2g:2.5+)

RESOLVED FIXED
feature-b2g 2.5+

People

(Reporter: jyeh, Assigned: jyeh)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(1 file)

When install apps with option key in marketplace, those apps cannot be found in the result of navigator.mozApps.getInstalled.

Install apps in preview window works as expected.
Flags: needinfo?(rchien)
Because navigator.mozApps.getInstalled() return apps according install origin and trigger "add app to apps" from option key in marketplace that will try to install app from system so invoking navigator.mozApps.getInstalled() from marketplace app will not return that installed app.

we can workaround it by installing app while pressing option key as in preview window does.

Luke, do you have a better idea?
Flags: needinfo?(rchien) → needinfo?(lchang)
As our offline discussion, I don't think installing apps every time is a good solution since there may be  performance issues. Another question is: do we really need to show the "installed" status of Apps (since we can't get the same status from Websites)?

For launching behavior, I would suggest that marketplace can install it itself in the handler of "contextmenu" event and system app needs to pause the preview mechanism when a install request follows a specific "contextmenu". However, I don't think it's a better solution though.
Flags: needinfo?(lchang)
Agree. I'm going to take over it and patch will be submitted soon.
Assignee: nobody → rchien
Status: NEW → ASSIGNED
Let's try to fix bug 1218289 and avoid hack from Gaia. Joseph, please use this bug do your part modification like comment 2 suggested. Thanks.
Assignee: rchien → jyeh
Summary: [TV][2.5] navigator.mozApps.getInstalled cannot get the apps installed by option key in marketplace → [TV][2.5] Hide price text in preview area
According to comment 2, we cannot get the install status of websites. Therefore, we decide to hide the price text for now as we are closed to the project deadline.
Hi Davor, this patch will hide the price text in the preview area. You can see more discussion on this bug.

Please let me know if you have any questions, thanks!
Attachment #8703931 - Flags: review?(dspasovski)
feature-b2g: --- → 2.5+
Whiteboard: [ft:conndevices]
Attachment #8703931 - Flags: review?(dspasovski) → review+
Landed in marketplace-tv-front-end master.

https://github.com/mozilla-b2g/marketplace-tv-front-end/commit/bd2396098d46a5178bda25e0be9b0617a8a04bf2
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
QA Whiteboard: [COM=TV::Web Apps]
You need to log in before you can comment on or make changes to this bug.