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

fix mouse events and adjust tests #4098

Closed
wants to merge 1 commit into from

Conversation

Ub1que
Copy link

@Ub1que Ub1que commented May 2, 2019

Hi, our team is developing web UI components and recently we faced an issue with cypress clicking on our widgets. While debugging the issue I found out that mouse events don't pass the "instanceof Event" check because the mouse event originated from the cypress window but not from the app window. You can check out example of our rich text editor. Right now clicking on any button on toolbar will not affect anything.

@CLAassistant
Copy link

CLAassistant commented May 2, 2019

CLA assistant check
All committers have signed the CLA.

@kuceb
Copy link
Contributor

kuceb commented May 2, 2019

Hi @Ub1que, thanks for this PR! This is a known issue and fixed in this PR (in review) #3030

We'll use the event constructor from the app window with these changes:

https://github.com/cypress-io/cypress/pull/3030/files#diff-7bde6af9f2fcd79039debba3b6103c50R564

@jennifer-shehane
Copy link
Member

@bkucera Can we close this PR then?

@kuceb
Copy link
Contributor

kuceb commented May 3, 2019

Closing in favor of #3030

@kuceb kuceb closed this May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants