Skip to content

Replace the legacy pointer input event handling with PointerPackets #288

Closed

Description

@FlutterIssues

Issue by jason-simmons
Tuesday Nov 10, 2015 at 00:33 GMT
Originally opened as flutter-team-archive/engine#2021


With this change we no longer need engine/core/events and its IDL bindings.


jason-simmons included the following code: https://github.com/flutter/engine/pull/2021/commits

Activity

  1. FlutterIssues commented on Nov 10, 2015

    @FlutterIssues
    Author

    Comment by chinmaygarde
    Tuesday Nov 10, 2015 at 01:18 GMT


    lgtm. I really like flushing all pending pointer events in one go.

    I will update the iOS platform stuff once this lands.

  2. FlutterIssues commented on Nov 10, 2015

    @FlutterIssues
    Author

    Comment by abarth
    Tuesday Nov 10, 2015 at 02:42 GMT


    LGTM

  3. FlutterIssues commented on Nov 10, 2015

    @FlutterIssues
    Author

    Comment by abarth
    Tuesday Nov 10, 2015 at 02:49 GMT


    You'll also need to update //services/sky, which is what builds the sky_viewer.mojo app that runs inside MojoShell. It's still using the legacy view manager, so you'll probably need to create pointer packets out of whatever input events the legacy view manager is using.

  4. FlutterIssues commented on Nov 10, 2015

    @FlutterIssues
    Author

    Comment by jamesr
    Tuesday Nov 10, 2015 at 05:32 GMT


    sky_viewer.mojo doesn't talk to the legacy view manager (or any other view manager), it talks to native viewport. The input events it gets are https://github.com/domokit/mojo/blob/master/mojo/services/input_events/interfaces/input_events.mojom instances from the NativeViewportEventDispatcher interface: https://github.com/domokit/mojo/blob/master/mojo/services/native_viewport/interfaces/native_viewport.mojom#L50

  5. FlutterIssues commented on Nov 10, 2015

    @FlutterIssues
    Author

    Comment by abarth
    Tuesday Nov 10, 2015 at 14:43 GMT


    @jamesr Yes, you're right. At some point, we should probably teach NativeViewportEventDispatcher (and eventually Mozart) about pointer packets.

  6. Hixie commented on Nov 11, 2015

    @Hixie
    Contributor

    @jason-simmons This is done now right?

  7. jason-simmons commented on Nov 11, 2015

    @jason-simmons
    Member

    This has landed.

  8. github-actions commented on Sep 7, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  9. locked as resolved and limited conversation to collaborators on Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions