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 unwanted trailing tab in chrome 53 #244

Merged
merged 2 commits into from
Oct 23, 2016
Merged

Conversation

doriancmore
Copy link
Contributor

@doriancmore doriancmore commented Sep 15, 2016

Closes #243

}

$element->postValue(array('value' => array($value)));
$this->blur($xpath);
Copy link
Member

Choose a reason for hiding this comment

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

The change event should be fired instead of blur to make tests pass.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@aik099
Copy link
Member

aik099 commented Sep 15, 2016

Also don't send PR from master branch, because you'll be unable to send 2 PR in parallel to same project.

@aik099
Copy link
Member

aik099 commented Sep 16, 2016

One test still fails. However I'm unsure if this is related to made changes or just to Firefox version updated on Travis CI, that doesn't play nice with Selenium Driver version that is used.

@aik099
Copy link
Member

aik099 commented Sep 16, 2016

For me latest stable Selenium Driver (2.53.0) + latest stable Firefox (48.0.1) doesn't work at all. When running test suite on Chrome I'm getting same " error.

@aik099
Copy link
Member

aik099 commented Sep 16, 2016

I've created #245 issue for that failing test because it's not related.

@aik099
Copy link
Member

aik099 commented Sep 16, 2016

Otherwise change looks good if it solves problem you're having in both Chrome and Firefox.

Copy link

@kwoxer kwoxer left a comment

Choose a reason for hiding this comment

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

Works for me in Chrome 53.0.2785.101

@sumthief
Copy link

Faced with the same problem on my current project. This solution works for me in Chrome 53.0.2785.116 (64-bit).

Copy link

@cocciagialla cocciagialla left a comment

Choose a reason for hiding this comment

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

Worked for me too

@aik099
Copy link
Member

aik099 commented Oct 22, 2016

The build failure is due different problem described in #245.

Otherwise tests are passing and I think this one is ready to be merged.

@aik099
Copy link
Member

aik099 commented Oct 23, 2016

Thanks @adorin , merging.

individual-it added a commit to owncloud/core that referenced this pull request May 18, 2017
this is needed to get this PR in minkphp/MinkSelenium2Driver#244
That fixes an issue with typing into autocomplete boxes
individual-it added a commit to owncloud/core that referenced this pull request May 18, 2017
need to get this PR in: minkphp/MinkSelenium2Driver#244 
It fixes an issue with typing into autocomplete boxes
individual-it added a commit to owncloud/core that referenced this pull request May 19, 2017
need to get this PR in: minkphp/MinkSelenium2Driver#244 
It fixes an issue with typing into autocomplete boxes
@hchonov hchonov mentioned this pull request Jun 15, 2018
@aik099 aik099 mentioned this pull request Sep 10, 2021
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.

5 participants