Skip to content

feat(dropdown): change forceSelection default to false #1747

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

lubber-de
Copy link
Member

@lubber-de lubber-de commented Nov 3, 2020

Description

This PR changes default value for forceSelection to false, because if you click into a dropdownfield and click outside again, the first dropdown value will be selected automatically which is probably not wanted in most cases so it shouldn't be the default.

If a select HTML field to be converted into a FUI dropdownfield got a required attribute, the forceSelection setting will be set to true again automatically

As this is a breaking change it is supposed to be part of 2.9

Closes

#1742

@lubber-de lubber-de added type/feat Any feature requests or improvements lang/javascript Anything involving JavaScript state/awaiting-reviews Pull requests which are waiting for reviews tag/breaking-change Any pull request which is waiting for a breaking change release labels Nov 3, 2020
@lubber-de lubber-de added this to the 2.9.x milestone Nov 3, 2020
Copy link
Contributor

@exoego exoego left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changing this default sounds reasonable.

@lubber-de lubber-de added the state/awaiting-docs Pull requests which need doc changes/additions label Nov 4, 2020
Copy link
Contributor

@prudho prudho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ko2in ko2in left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@lubber-de lubber-de added state/on-hold Issues and pull requests which are on hold for any reason and removed state/awaiting-reviews Pull requests which are waiting for reviews labels Nov 4, 2020
@lubber-de lubber-de changed the title feat(dropdown): change forceSelection default feat(dropdown): change forceSelection default to false Aug 9, 2021
@lubber-de lubber-de merged commit ed58b7f into fomantic:develop Aug 9, 2021
@lubber-de lubber-de deleted the feat/dropdown/forceSelectionFalseDefault branch August 9, 2021 15:26
@lubber-de lubber-de removed the state/on-hold Issues and pull requests which are on hold for any reason label Aug 9, 2021
@lubber-de
Copy link
Member Author

Docs added by fomantic/Fomantic-UI-Docs#294

@lubber-de lubber-de added state/has-docs A issue/PR which requires documentation changes and has the corresponding PR open in the docs repo and removed state/awaiting-docs Pull requests which need doc changes/additions labels Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/javascript Anything involving JavaScript state/has-docs A issue/PR which requires documentation changes and has the corresponding PR open in the docs repo tag/breaking-change Any pull request which is waiting for a breaking change release type/feat Any feature requests or improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants