search

DimitarChristoff / Modal

Modal Window and Bootstrap for mootools
http://jsfiddle.net/dimitar/GGAa5/
4 stars 2 forks source link

Can not have a <span> in a <a> #2

Closed Shine-neko closed 12 years ago

Shine-neko commented 12 years ago

The modal can not select a <a> with a <span> (eg <a> <span> foo </span> <a/>)

DimitarChristoff commented 12 years ago

er yeah. i can see that - reliance on e.target here is wrong in the delegation as that is the span and the span has no properties to build from. looking into it.

DimitarChristoff commented 12 years ago

fixed in b8443ec34071b392a7fd58b0b67ec6433957b253

DimitarChristoff commented 12 years ago

actually, a better / faster fix is this: 1ee7b0041bed74573614e3dc53e8cda10af64025 - thanks for the bug report.

Shine-neko commented 12 years ago

thanks :)