Opened 14 years ago
Closed 12 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 Changed 14 years ago by
Changed 14 years ago by
| Attachment: | 11684-dismissAddAnotherPopup-xhtml.diff added |
|---|
comment:2 Changed 14 years ago by
| milestone: | → 1.2 |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:3 Changed 14 years ago by
| 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 Changed 12 years ago by
| Severity: | → Normal |
|---|---|
| Type: | → Bug |
comment:5 Changed 12 years ago by
| 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