Provide flow diagrams for input management

RESOLVED FIXED in 2.1 S9 (21Nov)

Status

Firefox OS
Gaia::System::Input Mgmt
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mnjul, Assigned: mnjul)

Tracking

(Blocks: 1 bug)

unspecified
2.1 S9 (21Nov)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [p=2])

Attachments

(1 attachment)

After bug 1093537 lands, we're mostly done with the current iteration of (big) input management refactoring. Why not draw some diagrams for input management flows?
On this.
Assignee: nobody → jlu
Whiteboard: [p=2]
Target Milestone: --- → 2.1 S9 (21Nov)
Created attachment 8523794 [details] [review]
Patch (PR @ GitHub)

Tim, could you check this? I believe the flows-logics of the diagrams themselves are pretty fine (they're just "codify"ing the codes) and we don't need to involve Alive here?

With that said, of course, style-related comments are welcome.

Also, I do have two questions -- what is usually the best place to host the images? and what do we usually do for long URLs that break the linter limit?

Thanks!
Attachment #8523794 - Flags: review?(timdream)
Comment on attachment 8523794 [details] [review]
Patch (PR @ GitHub)

Let's do this on MDN or Wiki instead. An approach patch would be a link to these documentations instead of Github.

I have not yet look at the diagrams.
Attachment #8523794 - Flags: review?(timdream)
Comment on attachment 8523794 [details] [review]
Patch (PR @ GitHub)

Files are now hosted on Mozilla Wiki.
Attachment #8523794 - Flags: review?(timdream)
Comment on attachment 8523794 [details] [review]
Patch (PR @ GitHub)

As the diagram explains the current state of the code (not the eventual, ideal arch we want to achieve), I have no opinion on it.

Let's point people to 
https://wiki.mozilla.org/Gaia/System/InputManagement instead of the actual images because you do not want to submit a patch every time the diagram changes :)

Also, SVG is better than PNG if WikiMO accept that.
Attachment #8523794 - Flags: review?(timdream) → review+
Just checked and the wiki seems to support svg uploading. Let me see...
Master: https://github.com/mozilla-b2g/gaia/commit/d93eb7854aa269ab3aed5f851b8b01c9c1a07c7f

Note: I landed the patch as soon as Li went green as this is just comments change.

Diagrams are on that wiki page too. (it seems to have trouble rendering svn previews so I'm using PNG as thumbnail and link target as SVG).
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.