Skip to content
This repository has been archived by the owner on Nov 27, 2020. It is now read-only.

Manipulate dialog boxes #266

Closed
wants to merge 4 commits into from

Conversation

znz
Copy link

@znz znz commented Feb 28, 2013

This patch will support to get messages (of window.alert(), window.confirm(), and window.prompt()) and to set responses (for window.confirm() and window.prompt()) using onAlert, onConfirm, and onPrompt of phantomjs.

@jonleighton
Copy link
Contributor

Hi @znz, thanks for the patch.

Ideally I'd like there to be an officially blessed API from Capybara for doing this. Would you be up for working on getting that into Capybara? (cc @jnicklas)

If we don't have an officially blessed API, I think we should at least have the same API as what capybara-webkit has implemented.

Thanks

@jnicklas
Copy link

jnicklas commented Mar 3, 2013

Yes, official API in Capybara is a good idea :)

@gongo
Copy link
Contributor

gongo commented Mar 23, 2013

👍

@gongo
Copy link
Contributor

gongo commented Mar 26, 2013

Good patch! Anyway, implement the same API of capybara-webkit?

@mikepack
Copy link
Contributor

@asross
Copy link

asross commented Apr 16, 2013

Any updates? Sort of waiting for this feature :)

@mikepack
Copy link
Contributor

@asross has not been merged. Trying to determine if headless browsers can support retroactive notification responses. teamcapybara/capybara#1037

@asross
Copy link

asross commented Apr 16, 2013

@mikepack Cool. Thanks for your / everyone else's work on this.

@jonleighton
Copy link
Contributor

I'm going to close this PR for now as it's not going to be merged in its current state. Hopefully everyone here will keep up the good work to develop an official API though, which we can then get into Poltergeist. Thanks.

@gongo
Copy link
Contributor

gongo commented Jul 3, 2014

🎉 Merged Capybara's official API of alert/confirm teamcapybara/capybara#1322

@yaauie
Copy link
Contributor

yaauie commented Jul 3, 2014

@gongo yay!

Does anyone want to drive the feature to move us to the newly-merged APIs?

@mikepack
Copy link
Contributor

mikepack commented Jul 3, 2014

Cheers!!! 🍺

@gongo
Copy link
Contributor

gongo commented Jul 3, 2014

@yaauie

Does anyone want to drive the feature to move us to the newly-merged APIs?

I'll take it. ok?

@yaauie
Copy link
Contributor

yaauie commented Jul 3, 2014

@gongo 👍

@marnen
Copy link

marnen commented Jul 3, 2014

@gongo Great! Is there a new pull request?

@gongo
Copy link
Contributor

gongo commented Jul 3, 2014

@marnen

Yes! I will make new PR at a later date.

@yaauie
Copy link
Contributor

yaauie commented Jul 3, 2014

@marnen: @gongo just agreed to tackle this ~12 hours ago (and I doubt it's the highest priority in @gongo's life, especially since it's 3:45am UTC+9); please be patient 😄

@marnen
Copy link

marnen commented Jul 3, 2014

@yaauie No impatience meant. I was asking if @gongo had been working on an
older existing pull request, not whether he'd suddenly made a pull
request from scratch at 3:45 in the morning. :)

On Thursday, July 3, 2014, Ryan Biesemeyer notifications@github.com wrote:

@marnen https://github.com/marnen: @gongo https://github.com/gongo
just agreed to tackle this ~12 hours ago (and I doubt it's the highest
priority in @gongo https://github.com/gongo's life, especially since
it's 3:45am UTC+9); please be patient [image: 😄]


Reply to this email directly or view it on GitHub
#266 (comment)
.

Marnen Laibow-Koser
marnen@marnen.org
http://www.marnen.org

Sent from Gmail Mobile

@gongo
Copy link
Contributor

gongo commented Jul 3, 2014

@yaauie @marnen

Thanks. After taking a sleep well 😴 , I will take little by little 💪

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

Successfully merging this pull request may close these issues.

None yet

8 participants