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 a few tests for IE11 #4317

Merged
merged 5 commits into from
Jun 21, 2018
Merged

Fix a few tests for IE11 #4317

merged 5 commits into from
Jun 21, 2018

Conversation

pekam
Copy link
Contributor

@pekam pekam commented Jun 21, 2018

descriptions in the commit messages


This change is Reviewable

@ZheSun88
Copy link
Contributor

:lgtm:


Reviewed 6 of 6 files at r1.
Review status: all discussions resolved, 0 of 1 LGTMs obtained, and 1 stale


Comments from Reviewable

@pekam pekam merged commit a4eccf6 into master Jun 21, 2018
@pekam pekam deleted the several-ie11-test-fixes branch June 21, 2018 11:40
gilberto-torrezan pushed a commit that referenced this pull request Jul 11, 2018
* Fix PaperInputIT by moving to the end of the field before typing

* Fix AttachExistingDomElementByIdIT by using tab instead of enter to commit the value change of an input field

* WebComponentsIT: Skip browser log API which doesn't exist for IE11

* Move ExceptionStackTraceIT to test-dev-mode because stack trace is not shown in production mode

* Use TB query instead of getInShadowRoot
gilberto-torrezan pushed a commit that referenced this pull request Jul 11, 2018
* Fix PaperInputIT by moving to the end of the field before typing

* Fix AttachExistingDomElementByIdIT by using tab instead of enter to commit the value change of an input field

* WebComponentsIT: Skip browser log API which doesn't exist for IE11

* Move ExceptionStackTraceIT to test-dev-mode because stack trace is not shown in production mode

* Use TB query instead of getInShadowRoot
@gilberto-torrezan gilberto-torrezan added this to the 1.0.1 milestone Jul 11, 2018
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.

3 participants