Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Currently, line 57 of webform/js/webform.element.other.js consists of the following:
var $select = $element.find('.form-type-select');
However, not all themes actually add the class "form-type-select"; Basic, for example, currently adds "form-select". Wouldn't it be more future-proof to just look for the select element instead?
var $select = $element.find('select');
Am I missing something obvious? That's what the checkbox and radio "other" fields do.
If not, I'll roll a patch.
Comments
Comment #3
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedMake sense to me. I committed the change. Thanks.