Created attachment 604722 [details] [diff] [review] patch This should do it, although I haven't tested it yet. I kept the 'data' attribute since fillPopup would need to escape the value if put directly in oncommand. The code handling _nostyle appears to be dead, so I removed it.
Assignee: nobody → dao
Status: NEW → ASSIGNED
Comment on attachment 604722 [details] [diff] [review] patch Nice cleanup! r+ assuming it works. :)
Attachment #604722 - Flags: review?(dolske) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
You need to log in before you can comment on or make changes to this bug.