modal-form icon indicating copy to clipboard operation
modal-form copied to clipboard

Published 20 hours ago • verified publisher icon ramblex

added edit article modal functionality

Open sunwooz opened this issue 12 years ago • 2 comments

This is my first fork/pull, not sure if I did it correctly. Thanks for the example! I made some changes so it uses modal if you press edit. It doesn't update the page asynchronously, it just updates and redirects to the articles_path.

sunwooz avatar Jan 26 '13 20:01 sunwooz

I'm glad you found some use with the example! It's a great idea to get the edit form as a modal too. I have a couple of comments to make on your changes which I'll add as line comments on your pull request.

ramblex avatar Jan 27 '13 10:01 ramblex

Btw, what would be the best way to attach the .modal click event on the articles soon after it is created? Right now, I'm guessing the event is only firing on older articles because it's wrapped in $(document).ready and the page needs to be refreshed.

sunwooz avatar Jan 27 '13 15:01 sunwooz