IE bug - onclick in repeat region

Question

So, it took me most of the day to track this down.
I have a repeat region that is setting an onclick and the
page was throwing a JS error in IE6 and IE7.
But, after it threw the error everything worked including the
onclick.
I finally figured out that it worked if I changed this:
onclick="alert({modelid});"
To this:
onclick="alert('{modelid}');"
I had to surround the spry variable notation with
single-quotes.
The reason I hadn't done this to begin with is that I have an
integer in that field, so it's unnecessary.
I'd be happy to send any Adobe employee the file that I
created to demonstrate the problem.
If the single quotes are always necessary, then that should
be prominent in the instructions.
If not (and they shouldn't be), then I look forward to this
being fixed in the .4 release.
Keep up the good work guys. Spry really is impressive (when
IE, in all it's suckitude, isn't murdering it)

Answer 1

Hi cf_addict,
This is a known problem, so you are absolutely correct that
it should be something we warn folks about up-front. Don and I have
had some discussions about putting together a common problems page
and posting it on labs and including it in the next version of
Spry.
For the curious, it looks like IE pre-compiles the JS in on*
event attributes, so it barfs on unquoted data references because
it thinks it's a syntax error. Quoting the data reference prevents
the problem, and because JS does a lot of auto-coercing of types,
things just work most of the time.
--== Kin ==--