Opened 17 years ago

Closed 13 years ago

#3110 closed Bug (wontfix)

ManyToMany filter_interface widget renders incorrectly in IE 6 when the field is in a fieldset with class collapse

Reported by: Adam Endicott <leftwing17@…> Owned by: xian
Component: contrib.admin Version: newforms-admin
Severity: Normal Keywords: nfa-someday
Cc: leftwing17@… Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: yes

Description

When I put a ManyToManyField with filter_interface=True inside a fieldset with class "collapse", it renders the left (available) portion of the widget correctly, but the right (chosen) part renders as a normal select widget with all currently chosen values as options. This causes the problem that when the object is saved, only the first (currently selected) existing chosen option is saved as the value for that field.

Interestingly, if I add a new value to the chosen side (pick one on the left, then click the right arrow button), it seems to cause the right side of the widget to reevaluate itself and render correctly.

I'm only seeing this when the widget is within a collapsed fieldset.

I'm using the 0.95 release.

Attachments (1)

screenshot.png (54.1 KB ) - added by Adam Endicott <leftwing17@…> 17 years ago.
screenshot example of the problem

Download all attachments as: .zip

Change History (16)

by Adam Endicott <leftwing17@…>, 17 years ago

Attachment: screenshot.png added

screenshot example of the problem

comment:1 by Simon G. <dev@…>, 17 years ago

Resolution: invalid
Status: newclosed

Can you confirm if this is still a problem on a current checkout?

comment:2 by Gary Wilson, 17 years ago

Resolution: invalid
Status: closedreopened

I don't think we should close tickets just because they haven't been confirmed.

comment:3 by Adam Endicott <leftwing17@…>, 17 years ago

Unfortunately this is pretty difficult for me to test on my current setup, I don't have a windows machine that can access anything I can set up for a quick test. I spent a few minutes trying to set something up testing on trunk, and didn't get it going.

I can confirm that in Django 0.96 it fails as described above in IE6, and works as it should in IE 7 (as well as Firefox).

Hopefully somebody with a windows setup can test it in trunk. This should be all the model needed to test it:

from django.db import models

class Foo(models.Model):
    name = models.CharField(maxlength=10)

class Thing(models.Model):
    name = models.CharField(maxlength=10)
    foos = models.ManyToManyField(Foo, filter_interface=True)
    class Admin:
        fields = ((None, {'fields': ('name',)}),
                  ('Foos', {'classes': 'collapse',
                            'fields': ('foos',)}),)

comment:4 by Adam Endicott <leftwing17@…>, 17 years ago

Cc: leftwing17@… added

comment:5 by ludo@…, 17 years ago

I can confirm this bug is still present with a trunk checkout.

comment:6 by James Bennett, 17 years ago

Owner: changed from nobody to xian
Status: reopenednew
Triage Stage: UnreviewedAccepted
Version: 0.95newforms-admin

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

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

Keywords: nfa-someday added

Problem reported against old admin, should not block merge of newforms-admin.

comment:8 by Marc Garcia, 16 years ago

This one looks like a duplicate of #1621.

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

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

#1621 and #7278 are dups of this.

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

Also verified this is still a problem with IE6 (but not IE7) on newforms-admin r7737.

comment:11 by Alex Gaynor, 16 years ago

Karen, dumb question, did you mean that it is confirmed working in IE7, or hasn't been checked?

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

Sorry, I see my comment could be interpreted either way. I did try IE7 and cannot recreate the problem there. It seems to affect only IE6.

comment:13 by Łukasz Rekucki, 13 years ago

Severity: normalNormal
Type: defectBug

comment:14 by Julien Phalip, 13 years ago

UI/UX: set

comment:15 by Tomek Paczkowski, 13 years ago

Easy pickings: unset
Resolution: wontfix
Status: newclosed

If it's IE6 only, then it's wontfix, as we don't support IE6 anymore in admin.

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