Red option - foreground or background

Manjaro-Pek / pacli

A simple and interative Bash Frontend for Pacman/Yaourt/Pacaur

GNU General Public License v2.0

44 stars 8 forks source link

Red option - foreground or background #20

Closed Chrysostomus closed 8 years ago

Chrysostomus commented 8 years ago

I'd like to use red foreground instead of red background on risky options. It is more readable on most configs. Anybody object?

excalibur1234 commented 8 years ago

as long as the dangerous/red options are still clearly visible, i do not have a problem with that.

i actually see another advantage of your proposed change: it would be easier to have a normal and red/dangerous option in one line.

papajoker commented 8 years ago

an idea (not sure) show section help before run dangerous?

excalibur1234 commented 8 years ago

i am skeptical. it is definitely great for beginners, but experienced users could feel patronized. pacli is aimed at experienced/intermediate users only.

the option being red should be warning enough.

if all of you guys like this idea, i recommend to implement it in the following form: whenever a red option is chosen, the terminal gets cleared and the help of the red option is shown. the user has to press [ENTER] to continue. This way, an experienced user can chose a red option and just press enter twice, because he already knows what the red option does.

papajoker commented 8 years ago

ok i make this help with press [Enter] for continue red command in new branch help its easy to disable :

in code source (view var $PARAMS)
pacli can use a local file $HOME/.config/paclirc set helpred=0

note: paclirc is ok for other user config ...

excalibur1234 commented 8 years ago

looks like this issue is resolved.

i have added paclirc to the readme.md file.