1105231 - [Stingray][smart-system] Apply visual to contextmenu

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[KM Lee [:rexboy] (inactivate after Sep. 22)]

We need to apply visual spec to contextmenu.
Things to do:

1. sense current focused cursor and change its style programmatically
2. remove selection border
3. apply visual spec

Comment 1

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Attachment: Patch

This patch applies visual spec to context menu, but with a temporary solution:
*  The icons of context menu are svg files. We need to change them to TTF fonts after they're ready.

And it also changes behavior of shared css a little bit:
*  each included css file under tv_shared/style is allowed to have one
   same-named directory. All contents under that are copied together with css.

I need the behavior above to bring some shared icons into app.

John may you review the patch? I need to confirm if you also feel this solution is ok.

Comment 2

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8529669 [details] [review]
Patch

I give r+ for the "JS" parts. As to css, I feel visual may have opinions on it. We should show the result to Scott and discuss with him.

We should not use ttf in this case because the menu tag is a formal tag in w3c spec. It only supports label for text and icon for image url. If we want to use icon-font, we may break this spec and introduce new attribute to it. This violates the thought of not segmenting web. IMO, we should ask visual to give us png in these cases.

Comment 3

[KM Lee [:rexboy] (inactivate after Sep. 22)]

After discussing with Scott, the buttons need to be scrollable if there are too many options.
I'll add it to the patch.

Comment 4

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Attachment: patch v2

I made several changes including:
- Use png rather than svg
- Change PNG color by invert (temporarily)
- use Scrollable so we can scroll context menu when there are too many items.
- Items are centered if they don't exceed screen width.

Since there are much changes John may you do the review again? you may just see the second commit. Thank you!

Comment 5

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8545761 [details] [review]
patch v2

Looks good to me. I would suggest we file a bug to create a type of smart-button for option menu and use it here.

Comment 6

[KM Lee [:rexboy] (inactivate after Sep. 22)]

https://github.com/mozilla-b2g/gaia/commit/d03cc5bfa746f5a09e02677fa2bafb0b5916285d

Thanks for reviewing!