search

magmax / python-inquirer

A collection of common interactive command line user interfaces, based on Inquirer.js (https://github.com/SBoudrias/Inquirer.js/)
MIT License
976 stars 97 forks source link

inquirer.Separator() missing #19

Open markfink opened 7 years ago

markfink commented 7 years ago

I could not find the separator. Am I doing sth wrong, or missing feature? inquirer.js provides two variants:

magmax commented 5 years ago

Python inquirer is not a rewrite for everything in inquirer.js, but accomplish the same idea.

If I've understood it correctly, the separator is just a string to be used between questions, haven't I?

AuHau commented 5 years ago

Not sure what is this about, hence I am closing this issue.

Feel free to reopen with more details on what is the issue or what kind of feature this should be.

Psychokiller1888 commented 2 years ago

Necromancer level 15

I stumbled upon this by searching for it too. I'm moving away from py-inquirer as it's broken with python 3.10 and won't be updated and your project looks fresher and and and...

Separators were

            inquirer.List(
                name = 'mainMenu',
                message = 'What would you like to do now?',
                choices = [
                    'Discover devices on network',
                    'Connect to a device',
                    #Separator(),
                    'Prepare your SD card',
                    'Change device\'s password',
                    'Set device\'s name',
                    'Install your sound device',
                    'Sound test',
                    'Install Alice',
                    #Separator(),
                    'Start Alice',
                    'Restart Alice',
                    'Stop Alice',
                    'Enable Alice service',
                    'Disable Alice service',
                    #Separator(),
                    'Update Alice',
                    'Update system',
                    'Upgrade system',
                    'Reboot device',
                    'Uninstall your sound device',
                    'Enable bug report for next session',
                    'Check Alice logs',
                    'Check system logs',
                    'Exit'
                ]
            )

This was producing the following:

 ❯ Discover devices on network
   Connect to a device
   ---------------
   Prepare your SD card
   Change device's password
   Set device's name
   Install your sound device
   Sound test
   Install Alice
   ---------------
   Start Alice
   Restart Alice
   Stop Alice
   Enable Alice service
   Disable Alice service
   ---------------
   Update Alice
   Update system
   Upgrade system
   Reboot device
   Uninstall your sound device
   Enable bug report for next session
   Check Alice logs
   Check system logs
   Exit

So the Separators are, in this case, in list group separation

vt-rcheng commented 2 weeks ago

please reopen, separator is super important for usability.

Cube707 commented 2 weeks ago

I am going to reopen this to see how much this is requested, please leave your support for this feature if you want it.

also here are the relevant docs: https://www.npmjs.com/package/inquirer#separator

Cube707 commented 2 weeks ago

Adding wontfix as I will not be actively working on implementing this