Allows user to choose the default colour #50
Conversation
|
Woah, I wasn't expecting this! The code is a lot shorter than what I was implementing too - I'll check the repo out asap :) |
| @@ -99,7 +99,29 @@ else | |||
| gerror | |||
| fi | |||
|
|
|||
| cp -rH files/"${style}"/Numix/* "${dir}"/Numix/ | |||
| read -p "Which folder colour do you want? " answer | |||
| if [[ "default,blue,green,grey,orange,pink,purple,red,yellow" == *$answer* ]]; then | |||
There was a problem hiding this comment.
If the user enters something like "grey,orange", then the script will look for files named "grey,orange*" and maybe crash.
If someone has a better idea to secure this, that would be cool.
There was a problem hiding this comment.
I have a plan for this similar to how we do it for style, so don't worry about that :) Gonna basically move the /files/colours to / and then split the files by colour. Then the check will be the same as it is for style, that the colour selected has a folder existing.
|
As far as I can tell, this is good to go! I'll make some modifications and clean it up to be consistent with the rest of the code, but congratulations: your implementations was a lot smarter than what I had in mind :P |
Allows user to choose the default colour
Let the user choose the folders default color.
The script asks the user for the color after the style and replaces the symbolic links.