Skip to content

Conversation

@hasangursoy
Copy link

No description provided.

@myahlGDCI
Copy link

It looks like that case is needed for compatibility with multi-selects. I agree it needs to be optimized though. Perhaps this instead? Note - untested, just off the top of my head.

$field.find('option:selected').each(function() {
    val += $(this).val();
});

@hasangursoy
Copy link
Author

val = $field.val(); will be enough for multiselects

@myahlGDCI
Copy link

Alright, thanks. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants