germsvel / phoenix_test

PhoenixTest provides a unified way of writing feature tests -- regardless of whether you're testing LiveView pages or static (non-LiveView) pages.
https://hex.pm/packages/phoenix_test
MIT License
170 stars 22 forks source link

Do not assume first option to be selected by default for multiple select #82

Closed soundmonster closed 4 months ago

soundmonster commented 4 months ago

Current behavior for a multiple select is copying the default behavior for single select: assume the first option to be selected by default. This is fine for single selects (that's how a "combo box" works after all, an "empty" options needs to be included explicitly at the top), but this is not how multiple select works, as nothing is selected by default and it's not safe to assume the first option to be desired.

This patch doesn't assume the first option in a multiple select to be selected by default.