Closed
Bug 657194
Opened 15 years ago
Closed 15 years ago
Use YUI's escapeHTML() instead of our custom one
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.2
People
(Reporter: reed, Assigned: reed)
References
Details
Attachments
(1 file)
|
916 bytes,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
Now that bug 649879 has landed, we can replace our custom escapeHTML code with actual YUI-supported code.
|
Assignee | |
Updated•15 years ago
|
Target Milestone: --- → Bugzilla 4.2
|
|
Assignee | |
Comment 1•15 years ago
|
||
grumble, Bugzilla lost my attachment.
|
|
Assignee | |
Comment 2•15 years ago
|
||
Attachment #532491 -
Flags: review?(mkanat)
|
||
Comment 3•15 years ago
|
||
Comment on attachment 532491 [details] [diff] [review]
patch - v1
Looks good to me. :-)
Attachment #532491 -
Flags: review?(mkanat) → review+
|
|
Assignee | |
Updated•15 years ago
|
Flags: approval?
|
|
||
Comment 4•15 years ago
|
||
Has this patch been tested?
|
|
||
Comment 6•15 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified js/field.js
Committed revision 7834.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•