Skip to content

Commit bf54e2b

Browse files
committed
Regroup multiselect options to make classifying pending feedback simpler. Make the control larger and resizable. Fixes ietf-tools#1854.
- Legacy-Id: 10593
1 parent 5e0632e commit bf54e2b

2 files changed

Lines changed: 13 additions & 3 deletions

File tree

ietf/nomcom/forms.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,15 @@ def __init__(self, *args, **kwargs):
3939
positions = Position.objects.get_by_nomcom(self.nomcom).opened().order_by('name')
4040
results = []
4141
for position in positions:
42-
nominees = [('%s_%s' % (position.id, i.id), unicode(i)) for i in Nominee.objects.get_by_nomcom(self.nomcom).not_duplicated().filter(nominee_position=position).select_related("email", "email__person")]
42+
accepted_nominees = [np.nominee for np in NomineePosition.objects.filter(position=position,state='accepted').exclude(nominee__duplicated__isnull=False)]
43+
nominees = [('%s_%s' % (position.id, i.id), unicode(i)) for i in accepted_nominees]
4344
if nominees:
44-
results.append((position.name, nominees))
45+
results.append((position.name+" (Accepted)", nominees))
46+
for position in positions:
47+
other_nominees = [np.nominee for np in NomineePosition.objects.filter(position=position).exclude(state='accepted').exclude(nominee__duplicated__isnull=False)]
48+
nominees = [('%s_%s' % (position.id, i.id), unicode(i)) for i in other_nominees]
49+
if nominees:
50+
results.append((position.name+" (Declined or Pending)", nominees))
4551
kwargs['choices'] = results
4652
super(PositionNomineeField, self).__init__(*args, **kwargs)
4753

@@ -671,7 +677,7 @@ def set_nomcom(self, nomcom, user, instances=None):
671677
if self.feedback_type.slug != 'nomina':
672678
self.fields['nominee'] = MultiplePositionNomineeField(nomcom=self.nomcom,
673679
required=True,
674-
widget=forms.SelectMultiple,
680+
widget=forms.SelectMultiple(attrs={'class':'nominee_multi_select','size':'12'}),
675681
help_text='Hold down "Control", or "Command" on a Mac, to select more than one.')
676682
else:
677683
self.fields['position'] = forms.ModelChoiceField(queryset=Position.objects.get_by_nomcom(self.nomcom).opened(), label="Position")

ietf/templates/nomcom/view_feedback_pending.html

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@
77

88
{% block subtitle %} - Feeback pending{% endblock %}
99

10+
{% block morecss %}
11+
.nominee_multi_select { resize: vertical; }
12+
{% endblock %}
13+
1014
{% block nomcom_content %}
1115
{% origin %}
1216
<h2>Feedback pending from email list</h2>

0 commit comments

Comments
 (0)