Skip to content

Repository navigation

onExit don't fire if we hover only the first pixel of the component #1176

Closed
Closed
onExit don't fire if we hover only the first pixel of the component#1176
Assignees
igordmn
Labels
bugSomething isn't workingdesktopinputTouch, mouse, keyboard input relatedp:highHigh prioritywait for build
Milestone
1.0

Description

@igordmn
Igor Demin (igordmn)
opened on Sep 16, 2021

The test fails on the commented code:

        window.sendMouseEvent(MouseEvent.MOUSE_PRESSED, x = 90, y = 50)
        window.sendMouseEvent(MouseEvent.MOUSE_RELEASED, x = 80, y = 50)
        awaitIdle()

        window.sendMouseEvent(MouseEvent.MOUSE_EXITED, x = 900, y = 500)
        awaitIdle()
        assertThat(onExits).isEqualTo(1) // fails here

Probably this is because we change pointerId on MOUSE_RELEASED (search pointerId += 1 in DesktopOwners)

The test is not merged yet

Activity

  1. Igor Demin (igordmn)
    added
    bugSomething isn't working
    inputTouch, mouse, keyboard input related
    desktop
    on Sep 16, 2021
  2. Igor Demin (igordmn)
    self-assigned this
    on Sep 16, 2021
  3. Igor Demin (igordmn)
    added this to the 1.0 milestone on Nov 1, 2021
  4. Igor Demin (igordmn)
    added
    p:highHigh priority
    on Nov 1, 2021
  5. Igor Demin (igordmn)
    added a commit that references this issue on Nov 17, 2021

    Remove pointerId from ComposeScene

    0115405

  6. igordmn commented on Nov 18, 2021

    @igordmn
    Igor Demin (igordmn)
    on Nov 18, 2021
    ContributorAuthor

    The fix was reverted, as it causes another issues.

  7. igordmn commented on Nov 18, 2021

    @igordmn
    Igor Demin (igordmn)
    on Nov 18, 2021
    ContributorAuthor

    The reproducer of the issue in a real application:

    2021-11-18-38.mp4
    1. hover the first pixel of the scrollbar (not the second one)
    2. press the mouse
    3. release the mouse
    4. move mouse to the left

    Result: the scrollbar is still hovered

  8. Igor Demin (igordmn)
    changed the title [-]Fix WindowInputEventTest.`catch mouse move` in compose sources[/-] [+]onExit don't fire if we hover only the first pixel of the component[/+] on Nov 18, 2021
  9. Igor Demin (igordmn)
    added a commit that references this issue on Nov 21, 2021

    Fix crash when we press right mouse button during dragging with left …

    4f47c26
  10. Igor Demin (igordmn)
    added
    wait for build
    on Nov 22, 2021

  11. akurasov commented on Nov 23, 2021

    @akurasov
    akurasov
    on Nov 23, 2021
    Contributor

    Fixed in 1.0.0-rc2

  12. akurasov
    closed this as completedon Nov 23, 2021
  13. Igor Demin (igordmn)
    added a commit that references this issue on Jan 14, 2022

    Events

    0593438

  14. 4 remaining items

  15. Igor Demin (igordmn)
    added a commit that references this issue on Feb 28, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    bbff51a
  16. Igor Demin (igordmn)
    added 2 commits that reference this issue on Mar 28, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    9ba813e

    Remove pointerId and isMousePressed states from ComposeScene

    cab75d8
  17. Oleksandr Karpovich (eymar)
    added a commit that references this issue on Apr 18, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    f070197
  18. Oleksandr Karpovich (eymar)
    added a commit that references this issue on Jun 2, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    66f2e13
  19. Oleksandr Karpovich (eymar)
    added a commit that references this issue on Jun 27, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    46540f8
  20. Igor Demin (igordmn)
    added 2 commits that reference this issue on Aug 18, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    7a61275

    Remove pointerId and isMousePressed states from ComposeScene

    dd15cb6
  21. Oleksandr Karpovich (eymar)
    added a commit that references this issue on Oct 26, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    e477127
  22. Oleksandr Karpovich (eymar)
    added a commit that references this issue on Nov 16, 2022

    Remove pointerId and isMousePressed states from ComposeScene

    f55fbf3
  23. Oleksandr Karpovich (eymar)
    added a commit that references this issue on Jan 13, 2023

    Remove pointerId and isMousePressed states from ComposeScene

    8109b66
  24. Ivan Matkov (MatkovIvan)
    added a commit that references this issue on May 10, 2023

    Remove pointerId and isMousePressed states from ComposeScene

    de073c1

  25. okushnikov commented on Aug 23, 2024

    @okushnikov
    okushnikov
    on Aug 23, 2024

    Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.

  26. JetBrains
    locked and limited conversation to collaborators on Dec 18, 2024
  27. Zahari Zahariev (harryzz)
    added a commit that references this issue on Jul 28, 2026

    Remove pointerId and isMousePressed states from ComposeScene

    4a05c2b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingdesktopinputTouch, mouse, keyboard input relatedp:highHigh prioritywait for build

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions