humaan / Modaal

An accessible dialog window library for all humans.
http://humaan.com/modaal/
MIT License
2.72k stars 183 forks source link

Dynamicly switch modaal type #49

Closed nivnoiman closed 8 years ago

nivnoiman commented 8 years ago

on click with class ".video" set type property to "video"?

does it possible?

thanks

danhumaan commented 8 years ago

Hi @nivnoiman, i'm not sure if i'm understanding correctly, but you can certainly trigger a video modal type using a class of video, for example the below markup would do the trick assuming you have a link with a class of video.

$('.video').modaal({
    type: 'video'
});

Also, see demo page for example, about half way down.

danhumaan commented 8 years ago

Hi @nivnoiman, Just checking if you had seen my reply above? I'm looking to clean up the repository a little by addressing issues and pull requests, and it would be good to get an idea if this is something we should or could action.

nivnoiman commented 8 years ago

yes, thanks

בברכת יום נעים,

ניב נוימן

Niv945@Gmail.Com niv945@gmail.com w _052-4011331 _w www.NivNoiman.com http://nivnoiman.com/

On Mon, Jul 25, 2016 at 11:32 AM, Dan Moore notifications@github.com wrote:

Hi @nivnoiman https://github.com/nivnoiman, Just checking if you had seen my reply above? I'm looking to clean up the repository a little by addressing issues and pull requests, and it would be good to get an idea if this is something we should or could action.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/humaan/Modaal/issues/49#issuecomment-234886570, or mute the thread https://github.com/notifications/unsubscribe-auth/AD5TNnvMr36DjThQTqZT9Wh4eZe6Ovrsks5qZHSogaJpZM4IrYuj .

danhumaan commented 8 years ago

Hey, sorry I should have been clearer with my question before.

Did my original reply help you and resolve any issue you might have had, or are you attempting to do something different with your Modaal that you wanted to ask us about?

nivnoiman commented 8 years ago

Actually not so much, but I created some function with jquery that does that.

Thanks anyway:)

‏בתאריך יום שני, 25 ביולי 2016, Dan Moore notifications@github.com כתב:

Hey, sorry I should have been clearer with my question before.

Did my original reply help you and resolve any issue you might have had, or are you attempting to do something different with your Modaal that you wanted to ask us about?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/humaan/Modaal/issues/49#issuecomment-234891830, or mute the thread https://github.com/notifications/unsubscribe-auth/AD5TNslp0aiaRv_5JtubGgMBx6Z5UWSZks5qZHiUgaJpZM4IrYuj .

בברכת יום נעים,

ניב נוימן

Niv945@Gmail.Com niv945@gmail.com w _052-4011331 _w www.NivNoiman.com http://nivnoiman.com/

danhumaan commented 8 years ago

No worries, and thanks for using Modaal! if there are any other questions, bugs or enhancements you can think of we'd love to hear them. Just open any new issues as they come up :)