Opened 11 years ago

Closed 11 years ago

Last modified 9 years ago

#1635 closed defect (fixed)

[patch] Related object popup fails for edit_inline objects in IE/Win

Reported by: Christopher Lenz <cmlenz@…> Owned by: Adrian Holovaty
Component: contrib.admin Version: 0.91
Severity: normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

… because IE/Win coughs on periods in the window name.

[716] fixed a similar issue, but the equivalent code in the functions showRelatedObjectLookupPopup() and dismissRelatedLookupPopup() was not updated.

Patch in a sec (actually against magic-removal, but should also apply to trunk AFAICT).

Attachments (1)

django_1635.diff (1.1 KB) - added by Christopher Lenz <cmlenz@…> 11 years ago.
Patch that fixes the issue by replacing every period with a triple underscore

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by Christopher Lenz <cmlenz@…>

Attachment: django_1635.diff added

Patch that fixes the issue by replacing every period with a triple underscore

comment:1 Changed 11 years ago by Adrian Holovaty

Status: newassigned

comment:2 Changed 11 years ago by Adrian Holovaty

Resolution: fixed
Status: assignedclosed

(In [3066]) Fixed #1635 -- Admin-site related-object popup no longer fails for edit_inline objects in IE/Win. Thanks, Christopher Lenz

comment:3 in reply to:  description Changed 9 years ago by anonymous

Replying to Christopher Lenz <cmlenz@gmx.de>:

… because IE/Win coughs on periods in the window name.

[716] fixed a similar issue, but the equivalent code in the functions showRelatedObjectLookupPopup() and dismissRelatedLookupPopup() was not updated.

Patch in a sec (actually against magic-removal, but should also apply to trunk AFAICT).

comment:4 Changed 9 years ago by James Bennett

(In [6474]) 0.91-bugfixes: Backport [3066] and some related changes. Refs #1635, #106.

Note: See TracTickets for help on using tickets.
Back to Top