-
Notifications
You must be signed in to change notification settings - Fork 43
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
Add helpers messagebox
, confirm
and prompt
to Drab.Browser
#93
Comments
With |
alert
and confirm
to Drab.Browseralert
, confirm
and prompt
to Drab.Browser
alert
, confirm
and prompt
to Drab.Browsermessagebox
, confirm
and prompt
to Drab.Browser
@grych I would like to contribute and help resolve this issue, could you provide more context on the helpers? |
HI @Blackjaguar18, appreciated! Please create pair of each of the functions, normal and bang - as it is done for the other functions in Drab.Modal. Please use |
What is the status of this issue? |
Hi @nikhilmore54, it is not finished yest |
Ok. Can you just brief me what is to be done?
…On Fri, 12 Apr 2019, 14:07 Tomek Gryszkiewicz, ***@***.***> wrote:
Hi @nikhilmore54 <https://github.com/nikhilmore54>, it is not finished
yest
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#93 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AB3oek0fuTTqQYChjKOpIldgDcPM0ggEks5vgEXBgaJpZM4TEunA>
.
|
Hi @nikhilmore54, appreciated! "this is a simple wrapper around javascript alert, confirm and prompt. Please name function calling JS alert - messagebox/3, as it should not be confused with Drab.Modal.alert. Please create pair of each of the functions, normal and bang - as it is done for the other functions in Drab.Modal. Please use timeout: :infinite as default option in exec_js - we want to hold on processing while displaying an alert. Of course timeout should be overritable." |
This is a good, easy entry point to contribute!
The text was updated successfully, but these errors were encountered: