1121316 - [System2][Screen cleanup] Move ActionMenu into SystemDialog or AppWindow

Status: RESOLVED FIXED

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

Description

[(inactive after 6/18) Alive Kuo [:alive]]

ActionMenu is an orphan module now - it is having its own resize / home policy. The problem is we want to deprecate any #screen.locked rule consumer and it is one of them. The better way to maintain the sizing is having a manager to control the size in javascript instead of touching screen element.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] albertopq:1121316-action-menu-refactor > mozilla-b2g:master

Comment 2

[Alberto Pastor [:albertopq]]

Comment on attachment 8598092 [details] [review]
[gaia] albertopq:1121316-action-menu-refactor > mozilla-b2g:master

Still tests to be added, but Alive, could you please take a first look to see if I understood what you meant correctly? Thanks!

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8598092 [details] [review]
[gaia] albertopq:1121316-action-menu-refactor > mozilla-b2g:master

WOW, thanks for doing this! I left some comments but basically it's nice.

About the naming: I think if we have ActionMenu(which is not a dialog), we should rename SystemDialogManager to SystemUserInterfaceManager or whatever afterwards since it's not only managing dialogs but also menus.

Alos, we could remove the ActionMenu hijack in HierarchyManager if it's now managed by SystemDialogManager which is already under HierarchyManager.

One thing I am worry about is what SystemDialog and ActionMenu conflict - for example, is there a case that SIMPINDialog and ActivityMenu will show at the same time..? But we could refine this later, I guess.

Comment 4

[Alberto Pastor [:albertopq]]

Comment on attachment 8598092 [details] [review]
[gaia] albertopq:1121316-action-menu-refactor > mozilla-b2g:master

Fixed the tests and the naming comments you left.

I tried the case you were mentioning (SIM dialog when an activity action menu is visible) and the SIM is shown in top of it, so it should work fine.

Thanks!

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8598092 [details] [review]
[gaia] albertopq:1121316-action-menu-refactor > mozilla-b2g:master

Well done, thanks for the work!

Comment 6

[GitHub Autolander Bot]

Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/703b21afffb5d6c8f877d9aeb0d93525e8fd1642