Opened 15 years ago
Closed 14 years ago
#11684 closed Bug (duplicate)
dismissAddAnotherPopup fails with XHTML
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
dismissAddAnotherPopup looks at an element's .nodeName case-sensitively. This fails with XHTML. The attached patch fixes this.
Attachments (1)
Change History (6)
comment:1 by , 15 years ago
by , 15 years ago
| Attachment: | 11684-dismissAddAnotherPopup-xhtml.diff added |
|---|
comment:2 by , 15 years ago
| milestone: | → 1.2 |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:3 by , 15 years ago
| milestone: | 1.2 |
|---|
This is part of the larger issue of dealing with varying output formats, and so is probably best dealt with in a general fashion by a future release, rather than piecemeal in 1.2.
comment:4 by , 14 years ago
| Severity: | → Normal |
|---|---|
| Type: | → Bug |
comment:5 by , 14 years ago
| Easy pickings: | unset |
|---|---|
| Resolution: | → duplicate |
| Status: | new → closed |
This is actually a dupe of #5704.
Note:
See TracTickets
for help on using tickets.
See also #11683