Closed Bug 1122453 Opened 10 years ago Closed 7 years ago

Re-organize the APIs in ViewManager

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: rudyl, Unassigned)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1115247 +++ See Bug #1115247 comment 2, we need to re-organize the APIs that are exposed by viewManager. - One thing I remember is that we should not return an instance of AlternativesCharMenuView from the viewManager. - Another one is LayoutRenderingManager should be the only client of viewManager. Not sure if these are all that we can do in this, but will try to get started.
Assignee: nobody → rlu
Status: NEW → ASSIGNED
Attached file WIP (deleted) —
Tim, This is a WIP to refine the APIs, please help give any advice if you could help. Thanks. Please help look at this commit, since the previous patch has not landed yet. https://github.com/RudyLu/gaia/commit/37c3368a8f8a9db7b4fac2a8aab4e95dcd3f90b2
Attachment #8551167 - Flags: feedback?(timdream)
Comment on attachment 8551167 [details] WIP There are other stuff as well, for example the responsibility and APIs between rendering manager and view manager are not clear; the draw() function takes a callback which is the only place we take callback (all other callbacks are converted to promises already). There is also a problem we accidentally chain the requestAnimationFrame() to the promise chain and actually waits for the rendering before showing the keyboard. The list goes on if you understand the code enough. Please be opinionated and use your judgement to propose fixes. I really don't have the time to find a explicit list for you, unfortunately. If you really can't do it, you should probably work on Emoji layout first after bug 1115247.
Attachment #8551167 - Flags: feedback?(timdream)
Assignee: rlu → nobody
Status: ASSIGNED → NEW
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: