Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Is there any use for inline_selections anymore? #172

Closed
pksunkara opened this issue Feb 13, 2022 · 2 comments · Fixed by #277
Closed

Is there any use for inline_selections anymore? #172

pksunkara opened this issue Feb 13, 2022 · 2 comments · Fixed by #277
Labels
c-theme Context: Theme
Milestone
0.11.0

Comments

@pksunkara
Copy link
Collaborator

No description provided.

@pksunkara pksunkara mentioned this issue Feb 13, 2022
Open
@pksunkara pksunkara added this to the 0.11.0 milestone Feb 13, 2022
@pksunkara pksunkara added the c-theme Context: Theme label Feb 13, 2022
@Gordon01
Copy link
Contributor

Gordon01 commented Sep 5, 2023

No, looks like there is none. format_multi_select_prompt_selection() is always print selections divided by comma.
ColorfulTheme.inline_selections never set to false.

@pksunkara
Copy link
Collaborator Author

Agreed. Feel free to send a PR if you have time.

@Gordon01 Gordon01 mentioned this issue Sep 6, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c-theme Context: Theme
Projects
None yet
Milestone
0.11.0
Development

Successfully merging a pull request may close this issue.

Remove ColorfulTheme.inline_selections Gordon01/dialoguer
2 participants
@pksunkara @Gordon01