fix: handle remote_instructions field correctly on interim request form

Author: jennifer-richards

ietf/meeting/forms.py

@@ -285,6 +285,10 @@ def __init__(self, *args, **kwargs):
             choices.append(('meetecho', 'Automatically create Meetecho conference'))
         choices.append(('manual', 'Manually specify remote instructions...'))
         self.fields['remote_participation'].choices = choices
+        # put remote_participation ahead of remote_instructions
+        field_order = [field for field in self.fields if field != 'remote_participation']
+        field_order.insert(field_order.index('remote_instructions'), 'remote_participation')
+        self.order_fields(field_order)
 
     def clean_date(self):
         '''Date field validator.  We can't use required on the input because

ietf/static/js/meeting-interim-request.js

@@ -254,11 +254,11 @@ const interimRequest = (function() {
 
             switch (elt.value) {
             case 'meetecho':
-                remoteInstructions.closest('.form-group').hide();
+                remoteInstructions.closest('.row').hide();
                 break;
 
             default:
-                remoteInstructions.closest('.form-group').show();
+                remoteInstructions.closest('.row').show();
                 break;
             }
         }