Ticket #2184 (closed defect (bug): fixed)
newCatAddIn Javascript function tries to append a 'p' into a 'p' which causes an error in Konqueror.
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Administration | Version: | 2.0 |
| Severity: | major | Keywords: | javascript admin |
| Cc: |
Description
newCatAddIn Javascript function, which allows you to add categories via ajax, tries to append a paragraph (p) into a paragraph (p). Which causes an DOM error in Konqueror.
The simplest fix is to change the ajaxcat element to a 'span' as in the diff I will submit.
Attachments
-
admin-header.diff.txt
(464 bytes) -
added by MattRead 6 years ago.
- fix ajaxcat
-
admin-header2.diff.txt
(806 bytes) -
added by MattRead 6 years ago.
- fix ajaxcat and regex
Change History
-
attachment
admin-header.diff.txt
added
Well there was another error after fixing that one. one of the RegEx that strips out line endings has the \l escape character. without the \l it works great, and it seems, from what I can find, that no browser supports \l anyway. So I submit a new diff that fixes both errors. Sorry ... should have done this the first time :)
admin-header2.diff.txt is the one!
[3323] is related, and seems to fix it, but doesn't fix it (got changed back maybe?) ... and it introduces the new RegEx error in Konq. But all is still fixable by the diff. :)
- Status changed from new to closed
- Resolution set to fixed
fix ajaxcat