Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed mouse dispatcher #6429

Merged
merged 3 commits into from
Jun 11, 2024
Merged

fixed mouse dispatcher #6429

merged 3 commits into from
Jun 11, 2024

Conversation

Moerliy
Copy link
Contributor

@Moerliy Moerliy commented Jun 11, 2024

Describe your PR, what does it fix/add?

dispatcher issue that came up with the merge of #6358

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

-/-

Is it ready for merging, or does it need work?

needs testing but the error seems gone

@nonetrix
Copy link

oopsies
roblox

@gksudolol
Copy link

this busts the mouse completely :(

@Moerliy
Copy link
Contributor Author

Moerliy commented Jun 11, 2024

yeah I noticed it xD

@Moerliy
Copy link
Contributor Author

Moerliy commented Jun 11, 2024

sitting on it but my test setup is kind f'ed up so a godda do some pushing and pulling

@Moerliy
Copy link
Contributor Author

Moerliy commented Jun 11, 2024

@gksudolol can you test it again. Should work now

Copy link

@JoelleJS JoelleJS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue is fixed for me

@gksudolol
Copy link

  • OP is Jason Bourne?

Jesus Christ

Tested, resolved

Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should stop reviewing MRs barely alive. Dunno how I missed this

@vaxerski vaxerski merged commit 8412ffc into hyprwm:main Jun 11, 2024
11 checks passed
@Moerliy
Copy link
Contributor Author

Moerliy commented Jun 11, 2024

I don't know how I even managed to delete that line thb O_o

@nonetrix
Copy link

Fixed for me :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants