-
-
Notifications
You must be signed in to change notification settings - Fork 30
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.
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
Formatierung der Infos bei der Installation und besseres Feedback beim Fortschritt #2
Conversation
🏷️ Version 1.0.2
Es wird jetzt deutlicher gemacht., was genau gerade passiert. Vorher wurde nur mitgeteilt, das etwas fehlt. Jetzt wird jeder Abschnitt gezeigt (Überprüfung). Zudem wurde log zu echo ersetzt und es wurden einige Farb- und Sonderzeichenformatierungen hinzugefügt, damit es nicht so langweilig aussieht.
Der Prozess wurde von der Sicht des Nutzers komplett überarbeitet. Man bekommt jetzt mehr Feedback, was genau gerade passiert. Einige Sonderzeichen (Unicodes) wurden hinzugefügt und die Hinweise wurden überarbeitet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich bin auch der Meinung, dass das alles mit Emojis besser aussieht. Noch die 4 kleinen Änderungen hier und da und dann kann ich das ganze mergen. Danke schonmal 👀
install.sh
Outdated
echo -e "$GREEN-$NORMAL-$GREEN-$NORMAL$GREEN-$NORMAL$GREEN-$NORMAL$GREEN-$NORMAL$GREEN-$NORMAL$GREEN-$NORMAL$GREEN-$NORMAL$GREEN-$NORMAL" #multicolor | ||
echo -e "$GREEN✓ Installation abgeschlossen: $NORMAL MySpeed wurde unter $INSTALLATION_PATH installiert." | ||
echo -e "Die Weboberfläche findest du im Browser unter $BLUEhttp://localhost:5216$NORMAL." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier müsste man das Leerzeichen verschieben, weil sonst Bash das "http" noch als Variable erkennt
echo -e "Die Weboberfläche findest du im Browser unter $BLUEhttp://localhost:5216$NORMAL." | |
echo -e "Die Weboberfläche findest du im Browser unter$BLUE http://localhost:5216$NORMAL." |
install.sh
Outdated
if ! command -v curl &> /dev/null | ||
then | ||
log "Das Paket \"curl\" wurde nicht gefunden, wird aber benötigt. Es wird nun installiert..." | ||
echo -e "$YELLOWℹ\"curl\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier sollte noch ein Leerzeichen ergänzt werden, weil sonst die Anführungszeichen direkt neben dem Emoji stehen, was ein bisschen komisch aussieht 😅
echo -e "$YELLOWℹ\"curl\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." | |
echo -e "$YELLOWℹ \"curl\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." |
install.sh
Outdated
if ! command -v node &> /dev/null | ||
then | ||
log "Das Paket \"nodejs\" wurde nicht gefunden, wird aber benötigt. Es wird nun installiert..." | ||
echo -e "$YELLOWℹ\"node\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier das gleiche wie vorhin. Vielleicht wieder einfach ein Leerzeichen zwischen dem Emoji und den Anführungszeichen
echo -e "$YELLOWℹ\"node\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." | |
echo -e "$YELLOWℹ \"node\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." |
install.sh
Outdated
if ! command -v unzip &> /dev/null | ||
then | ||
log "Das Paket \"unzip\" wurde nicht gefunden, wird aber benötigt. Es wird nun installiert..." | ||
echo -e "$YELLOWℹ\"unzip\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier auch wieder das Problem mit den Anführungszeichen. Hier könnte auch wieder ein Leerzeichen rein, damit das ganze besser aussieht
echo -e "$YELLOWℹ\"unzip\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." | |
echo -e "$YELLOWℹ \"unzip\" ist nicht installiert.$NORMAL Die Installation wurde gestartet..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, jetzt passt alles :)
Die Texte wurden überarbeitet und es wurden Farben + Icons (Unicode) hinzugefügt.