-
Notifications
You must be signed in to change notification settings - Fork 10
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
Demo: Provide an option to take text input #21
Comments
Hello, I would like to work on this issue. |
Thanks, I will start working on this. |
Hey , Can i also work on this ? |
2 people together? No!! Let give some time to @asingh217 . If she doesn't finish it within a week or if there is no update then you can pick it. You can check more issues here
|
Ok , I will look for some other projects then. |
@asingh217 Let me know if there is an issue. I can help you test from my end. |
Thanks @ashishpatelcs , I need some suggestion on UI items like the
Please let me know how to do i share my code. Should i share the commit page link here from my fork. |
Here is your code changes: https://github.com/asingh217/shabdawali |
@asingh217 It would be better you can have it live on your GitHub pages so that I can see it working. Have you checked the code you've edited to work on your local machine? I have some doubts about it. Also, you can name the button like 'edit' or better you can use something like fontawesome or pencil like graphics. @amitguptagwl can enlighten us with some better ideas. Let's make it work. |
Here are the common fonts which are small in size and fulfill out need. Or you may simply use a button with text "edit text" @ashishpatelcs You can check her changes here: http://htmlpreview.github.io/?https://github.com/asingh217/shabdawali/blob/master/index.html |
@amitguptagwl Thank you. @asingh217 Your button and the modal box is working perfectly fine. |
Thanks, I have updated the title and button name. |
I think this issue is fixed. |
In the current implementation text is hard coded. We want to give an option so that user can try the demo with different text. For this;
The text was updated successfully, but these errors were encountered: