Skip to content

Unlimited select field options#2750

Merged
bobdenotter merged 1 commit intobolt:4.2from
JTNMW:4.2
Aug 11, 2021
Merged

Unlimited select field options#2750
bobdenotter merged 1 commit intobolt:4.2from
JTNMW:4.2

Conversation

@JTNMW
Copy link
Copy Markdown
Contributor

@JTNMW JTNMW commented Aug 4, 2021

Allows the number of options displayed in select fields to be set by the limit value in the contenttypes.yaml

Allows the number of options displayed in select fields to be set by the limit value in the contenttypes.yaml
@bobdenotter
Copy link
Copy Markdown
Member

Hi @JTNMW,

CI seems to be complaining about the line with optionsLimit, because it disapproves of CamelCase. I wonder if it's treated as case sensitive, or if we could just use optionslimit there too? 🤔

@JTNMW
Copy link
Copy Markdown
Contributor Author

JTNMW commented Aug 6, 2021

hi @bobdenotter ,

I followed the syntax given in the vue-multiselect documentation https://vue-multiselect.js.org/#sub-props. I'm not sure how strict these need to be followed. I'll give it a go in lower case and let you know if i still get the same results.

Copy link
Copy Markdown
Member

@I-Valchev I-Valchev left a comment

Choose a reason for hiding this comment

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

Looks solid, thanks @JTNMW.

If you would like to also introduce the autocomplete option in 4.2, I'd suggest checking here: https://github.com/bolt/core/pull/2727/files

A few things are missing in the current PR. It maybe best to not introduce this here and, instead, create a separate PR that makes the autocomplete work for 4.2 :-)

@bobdenotter
Copy link
Copy Markdown
Member

Thanks, @JTNMW 👍

Let's roll with this!

It maybe best to not introduce this here and, instead, create a separate PR that makes the autocomplete work for 4.2 :-)

If we feel we should backport fixes, let's do so in a separate PR!

@bobdenotter bobdenotter merged commit d6bb476 into bolt:4.2 Aug 11, 2021
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.

3 participants