add colors theme #1153
add colors theme #1153
Conversation
|
Ready for merging |
There was a problem hiding this comment.
Danke für das Colors Theme.
Zusätzlich zu den inline Kommentaren solltest Du bitte eine einheitliche Formatierung verwenden. Aktuell nutzt Du je nach Datei Tabs oder 2 bzw. 4 Leerzeichen zur Einrückung. Auch das Dateiende ("final new line") variiert. Am Besten nutzt Du eine .editorconfig sowie den in package.json definierten lint Befehl. Dadurch sollte auf allen Systemen eine einheitliche Formatierung herauskommen. Mit der VUE-Integration in den Linter würden dann auch Probleme wie das vorhandene v-if in v-for auffallen.
packages/modules/web_themes/colors/source/src/assets/js/mqttClient.ts
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/colors/source/src/assets/js/mqttClient.ts
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/colors/source/src/components/mqttViewer/MQTTViewer.vue
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/colors/source/src/components/mqttViewer/mqttClient.ts
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/colors/source/src/components/mqttViewer/mqttClient.ts
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/colors/source/src/components/powerGraph/PGMenu.vue
Show resolved
Hide resolved
|
Hallo Lutz, danke fürs Reviewen und das Feedback! Ich habe alle Punkte korrigiert. Ausserdem die ESLint-Konfig angepasst und alle Findings korrigiert, sowie prettier über alles laufen lassen. Ich hoffe mal, dass es jetzt passt :). |
|
Das sieht gut aus! |
Colors theme integriert.