Closed Bug 1233681 Opened 9 years ago Closed 9 years ago

[TV][2.5] Update contextmenu id to the top element

Categories

(Marketplace Graveyard :: General, defect, P2)

defect

Tracking

(blocking-b2g:2.5+)

RESOLVED FIXED
blocking-b2g 2.5+

People

(Reporter: Fischer, Assigned: jyeh)

References

Details

(Whiteboard: [partner-cherry-pick][ft:conndevices][partner-blocker])

Attachments

(1 file)

Panasonic TV dispatches the contextmenu event by the right-clicked mouse event with cursor position of (0, 0), so when trying to get contextmenu, it would take element determined by the mouse cursor position as focused element, not current element being navigated on by keys.
This results unable to get contextmenu data from element being navigated so no context would be present.
We have to update contextmenu id to the top element so contextmenu data could be found on TV.
[Blocking Requested - why for this release]:
blocking-b2g: --- → 2.5?
No longer depends on: 1227929
@Joseph,
Please look at this issue. We have to make some change on our TV marketplace.
Let's have a detailed face-to-face explanation to you when you are back to the office.
Thank you
Flags: needinfo?(jyeh)
Assignee: nobody → jyeh
Status: NEW → ASSIGNED
Flags: needinfo?(jyeh)
Hi Davor,

Instead of using multiple menu inside each app's icon, we turn up to use a single menu and update its label dynamically by javascript when an app is focused.

This is a compromise with Panasonic TV.


Please let me know if you have any questions about it, thanks :)
Attachment #8701342 - Flags: review?(dspasovski)
Comment on attachment 8701342 [details] [review]
jostw:josyeh/Marketplace/master/Bug1233681 > mozilla-b2g:master

Reassign review to Evelyn
Attachment #8701342 - Flags: review?(dspasovski) → review?(ehung)
Attachment #8701342 - Flags: review?(ehung) → review?(lchang)
blocking-b2g: 2.5? → 2.5+
Flags: needinfo?(lchang)
Flags: needinfo?(rexboy)
Comment on attachment 8701342 [details] [review]
jostw:josyeh/Marketplace/master/Bug1233681 > mozilla-b2g:master

r+ with a nit on GitHub. Thanks.
Attachment #8701342 - Flags: review?(lchang) → review+
Landed in marketplace-tv-front-end master

https://github.com/mozilla-b2g/marketplace-tv-front-end/commit/779dfc13dff2c0889e1355e77129624bfaa90724
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: