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

Use native WebDriver way of handling alerts #243

Closed
geb-bot opened this issue Jan 15, 2013 · 2 comments
Closed

Use native WebDriver way of handling alerts #243

geb-bot opened this issue Jan 15, 2013 · 2 comments

Comments

@geb-bot
Copy link

geb-bot commented Jan 15, 2013

Originally created by Marcin Erdmann.

This would greatly simplify the code in AlertAndConfirmSupport as well as would allow to avoid issues in situations where page is reloaded after an alert/confirm has been dismissed and where you cannot access the alert/confirm message.

@geb-bot geb-bot added the Bug label Apr 12, 2015
@geb-bot geb-bot added this to the 0.10.1 milestone Apr 12, 2015
@erdi erdi modified the milestone: 0.10.1 Jul 11, 2015
@erdi erdi added Improvement and removed Bug labels Jan 17, 2016
@erdi erdi added this to the 1.0 milestone Aug 24, 2016
@erdi erdi reopened this Oct 3, 2016
@erdi erdi removed this from the 1.0 milestone Oct 3, 2016
@erdi
Copy link
Member

erdi commented Oct 3, 2016

This has been reverted and reopened because some drivers seem to not implement the native way of handling alerts correctly or their implementation is not fully stable.

@erdi
Copy link
Member

erdi commented Jun 13, 2018

Given the incompatibilities between browsers in native alerts handling I think it's better to keep the compatibility layer Geb provides for the price of not being able to access the alert/confirm message when page is reloaded after an alert/confirm has been dismissed.

@erdi erdi closed this as completed Jun 13, 2018
@erdi erdi added the Won't fix label Jun 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants