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

Status: RESOLVED FIXED

(Marketplace Graveyard :: General, defect, P2)

Description

[Fischer [:Fischer]]

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.

Comment 1

[Fischer [:Fischer]]

[Blocking Requested - why for this release]:

Comment 2

[Fischer [:Fischer]]

@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

Comment 3

[Joseph Yeh [:joseph]]

Attachment: jostw:josyeh/Marketplace/master/Bug1233681 > mozilla-b2g:master

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 :)

Comment 4

[Joseph Yeh [:joseph]]

Comment on attachment 8701342 [details] [review]
jostw:josyeh/Marketplace/master/Bug1233681 > mozilla-b2g:master

Reassign review to Evelyn

Comment 5

[Josh Cheng [:josh]]

Hi Luke, Hi Rex,
Can you help review the patch while Evelyn is on PTO?
Thanks!

Comment 6

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8701342 [details] [review]
jostw:josyeh/Marketplace/master/Bug1233681 > mozilla-b2g:master

r+ with a nit on GitHub. Thanks.

Comment 7

[Joseph Yeh [:joseph]]

Landed in marketplace-tv-front-end master

https://github.com/mozilla-b2g/marketplace-tv-front-end/commit/779dfc13dff2c0889e1355e77129624bfaa90724