Add maxVisible option for select prompts

[elliot-nelson]

SUMMARY

Allow the "select" prompt to display a subset of the options at a time.

DETAILS

Changes to select

• optionsPerPage option added, similar to multiselect
• CHANGE - optionsPerPage now defaults to 10, which is a change in behavior if you had select lists with more than 10 elements. (I think this change is a net positive, but it probably requires at least a minor version bump.)

Changes to multiselect

• UI: scrolling through options when there are more options than can be displayed now shows small arrows indicators next to the options list at the top/bottom of the list. The hint text below the list has been removed.

Test changes

• Added a new tape file for utility unit tests, with a single entry (for entriesToDisplay).

PREVIEW

[elliot-nelson]

Reviewer note: after making this change, I realized that the multiselect already has this feature, although with different UI and different option name. (But, basically the same behavior otherwise).

I think before merging this I'd like to:

1. Change my implementation to match multiselect option name (optionsPerPage), for consistency.
2. Change multiselect UI to match mine (drop the "Scroll up or down to see more options verbiage", and replace it with the arrow indicators, which I think are more interactive and intuitive).

[terkelg]

Looks really good @elliot-nelson! Could be nice to have this as a part of the next release. I agree we should try to have a consistent API 👍

[elliot-nelson]

Updated preview image to show changes to select and multiselect.

[bbernstein]

I was looking at this for autocomplete around the same time you submitted this. I was more focused on minimal code changes, but this is probably a cleaner solution and looks better. I can cancel my pull and feel free to add this to the autocomplete versions of this. The arrowIndicator is a nice touch.

Since it's not merged yet. I have a version that does the same for autocomplete using your helper-methods. One concern with mine is that it removes the pages in autocomplete. I don't know how much the pages are preferred, but I have the earlier PR for one that just puts everyhing in page 1 but adds another option smooth. Anyway, I'm happy to contribute one of these versions once this is merged.

[elliot-nelson]

@bbernstein that looks great!

Personally, I think the pagination in the autocomplete should go away completely and just be scrolling, as then the experience for select, multi select, autocomplete, and autocompleMultiselect can all be the same.

(I like your "smooth" experience more than the pagination.)

[terkelg]

Thank you so much @elliot-nelson and @bbernstein. I agree, I think we should use this across all prompts instead of pagination. This provides a better user experience for sure. I appreciate the help. Can we choose/merge this and #227. I'm a bit busy this week, but I'll have some time to merge and release the next version over the weekend

[bbernstein]

I'll cancel what I had done earlier (at the same time as @elliot-nelson) and have made changes that will be based on this PR. Once this one is merged, I'll add my changes for a separate PR.

[terkelg]

@bbernstein this is now merged into master