Opened 18 years ago

Closed 16 years ago

#1621 closed defect (duplicate)

manytomany admin widget broken under IE6.0 when in collapsed group

Reported by: Olive Owned by: xian
Component: contrib.admin Version:
Severity: normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

when your uncollapse the first time then the right list show only one
entry (in fact it is a dropdown at this stage).

If you save without performing any other operation before, then only
the shown entry will saved in the database (others entries will be LOST
!).

I you move one or more entry from a list to the other then the normal
widget appearance is restored and a subsequent save operation works as
expected.

Change History (10)

comment:1 by James Bennett, 18 years ago

priority: highestnormal
Severity: blockernormal

I'll take a look at this tomorrow and see if I can't roll a fix into the pending Django/Dojo stuff. Meanwhile, I don't think this is really a "highest priority" or "blocker" level of bug if it's gone the entire time from Django's original development at World Online a couple years ago and through eight months of public poking and prodding without anyone else catching it.

comment:2 by anonymous, 18 years ago

Hi,
Thank you for taking care of this.
If I can use Django/Dojo stuff then it will no more a blocker for me.
Could you please tell me how to integrate Django/Dojo ? (I'm sittting on MR branch)

comment:3 by Olive, 18 years ago

Reading the Wiki page about Ajax I realised that some Dojo widgets are already use in Admin MR branch.
Is M2M a Dojo widget ?

comment:4 by Adrian Holovaty, 18 years ago

Olive: No. Dojo isn't being used for anything in Django yet.

comment:5 by Olive, 18 years ago

OK, for now I will avoid collapsable groups :_(
Thanks for all Adrian.

comment:6 by Gustavo Narea, 18 years ago

I added a line to django/contrib/admin/media/js/CollapsedFieldsets.js

        if (collapsed_seen) {
            // Expand all collapsed fieldsets when form is submitted.
            addEvent(findForm(document.getElementsByTagName('fieldset')[0]), 'submit', function() { CollapsedFieldsets.uncollapse_all(); });
        }
        CollapsedFieldsets.uncollapse_all();           // <----------------------------------------------
    },

I worked, but the page loads with the fieldsets uncollapsed now.

comment:7 by Gary Wilson <gary.wilson@…>, 17 years ago

Triage Stage: UnreviewedAccepted

comment:8 by James Bennett, 17 years ago

Owner: changed from nobody to xian

Reassigning to xian since he's doing JS stuff for newforms-admin.

comment:9 by Marc Garcia, 16 years ago

This one looks like a duplicate of #3110.

Could somebody check if that is true, and close as duplicate one?

comment:10 by Karen Tracey <kmtracey@…>, 16 years ago

Resolution: duplicate
Status: newclosed

This, #3110, and #7278 are all reporting the same problem. Confirmed still a problem with IE6 (but not IE7) on newforms-admin r7737. Keeping #3110 open since it's got the right release/tags.

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