On Wed, Feb 11, 2009 at 5:16 AM, Klaus Hartl <klaus.ha...@googlemail.com> wrote: > > Thanks for this information. I'm about to rewrite this stuff a bit > anyway due to some other bug, and I'll take your findings into > account. Thanks again.
No problem. I'll try again to get logged into Trac, create a new UID, if necessary, and add some of this to the bug listing. I'll remove my wild speculation, though ;-) > What I really wonder about is, that this code used to work actually. > This is why this whole logic with filtering an images in there... It > surely needs a proper unit test this time! I really wonder about that EM that's used to wrap the spinner option. It seems to me that it'd be simpler to just include that in the default text. That way, cleanup() could just switch everything inside the A. Unless there's some issue with storing the label data in the A.
