search

mootools / mootools-demos

Demos. For MooTools.
http://mootools.net/demos
MIT License
102 stars 61 forks source link

Fix to multi-pseudo demo #5

Closed asuth closed 13 years ago

asuth commented 13 years ago

Fixing multi-pseudo demo: pressing any character then enter produced weird results. Also using correct element, not the one that happened to be right

arian commented 13 years ago

Thanks for noticing. Simply changing text to value fixed to too.

asuth commented 13 years ago

umm, yes. But I fixed it to how it should be, using the element passed through by delegation. The demo happens to work because there's only one textarea therefore getElement references the same thing, but the true power of delegation is lost in the demo as it is shown now.

arian commented 13 years ago

yeah, you got a point there...

ibolmo commented 13 years ago

So then... is this ready to be merged?

On Sat, Jun 25, 2011 at 12:40 AM, asuth < reply@reply.github.com>wrote:

Fixing multi-pseudo demo: pressing any character then enter produced weird results. Also using correct element, not the one that happened to be right

Reply to this email directly or view it on GitHub: https://github.com/mootools/mootools-demos/pull/5

arian commented 13 years ago

Ok, merged / cherry-picked & deployed.

arian commented 13 years ago

Thanks for keeping us focused ^^

ibolmo commented 13 years ago

:D