Navigation

    Logo
    • Register
    • Login
    • Search
    • Recent
    • Tags
    • Unread
    • Categories
    • Unreplied
    • Popular
    • GitHub
    • Docu
    • Hilfe
    1. Home
    2. Deutsch
    3. ioBroker Allgemein
    4. [Kommentar] Style-Guide

    NEWS

    • Neues Video "KI im Smart Home" - ioBroker plus n8n

    • Neues Video über Aliase, virtuelle Geräte und Kategorien

    • Wir empfehlen: Node.js 22.x

    [Kommentar] Style-Guide

    This topic has been deleted. Only users with topic management privileges can see it.
    • apollon77
      apollon77 last edited by

      Bin gespannt

      1 Reply Last reply Reply Quote 0
      • Stabilostick
        Stabilostick last edited by

        Na da habe ich ja was angestellt…. :roll:

        Schade, das ich bis jetzt nix mit UI außer davor sitzen anfangen kann. Seufz.

        1 Reply Last reply Reply Quote 0
        • Jey Cee
          Jey Cee Developer last edited by

          Bitte erwartet nicht zu viel. Die Pläne sind gewaltig, aber genau so groß die Wissenslücken.

          Außerdem fehlt mir gerade leider die Zeit zum Programmieren.

          Gesendet von meinem m8 mit Tapatalk

          1 Reply Last reply Reply Quote 0
          • Bluefox
            Bluefox last edited by

            Eigentlich sollte es styleguide sein. Mindestens dachte ich dass eins gibt.

            Nun dürch die Änderungen, die MaterialCSS auch macht sind manche Beschriftungen nach unten gerutscht.

            Fakt list, die müssen oben sein und alles anderes ist Bug.

            Auch für Code gibt es Style Guide: Google JS Style.

            1 Reply Last reply Reply Quote 0
            • apollon77
              apollon77 last edited by

              Interessant … höre ich alles zum ersten mal :-))

              @Bluefox: Nicht hauen Bitte 😉

              Haben es halt nie enforced und wenn dann UI-Legasteniker wie ich an UIs rumpfuschen passiert sowas 😞

              1 Reply Last reply Reply Quote 0
              • Meistertr
                Meistertr Developer last edited by

                @Bluefox:

                Auch für Code gibt es Style Guide: Google JS Style. `
                Dann weiß ich nun was ich dringend mal lesen sollte..

                Edit: das ist aber ganz schön viel..

                Gesendet von meinem Handy

                1 Reply Last reply Reply Quote 0
                • AlCalzone
                  AlCalzone Developer last edited by

                  @Bluefox:

                  Auch für Code gibt es Style Guide: Google JS Style. `
                  2 Spaces indentation, ughhh :?

                  Naja, ich hab meinen eigenen Style "Guide" für meine Libs & Adapter. Recommended-Einstellungen von TSLint + https://github.com/AlCalzone/ioBroker.tradfri/blob/master/tslint.json. Das wird dann auch schön automatisch vom Editor umgesetzt bzw. angemeckert.

                  @Bluefox:

                  Eigentlich sollte es styleguide sein. Mindestens dachte ich dass eins gibt.

                  Nun dürch die Änderungen, die MaterialCSS auch macht sind manche Beschriftungen nach unten gerutscht.

                  Fakt list, die müssen oben sein und alles anderes ist Bug. `
                  Die Erfahrung hab ich auch gemacht bei einem Adapter. Da waren 1-2 Felder, die einfach nicht so wollten wie ich und anders aussahen als der Rest. Das hat schon 1h Google + etwas Tricksen gekostet bis alles normal aussah.

                  1 Reply Last reply Reply Quote 0
                  • AlCalzone
                    AlCalzone Developer last edited by

                    @apollon77:

                    Muss es wirklich eine IDE für Adapter-Entwicklung werden?

                    DIe Idee eines "Adapter-Generators" finde ich Bombe. `

                    Könnte eine Erweiterung des template-adapters werden und so in die Richtung von create-react-app gehen. Eine Anwendung, die man einmalig z.B. per npx ausführt und aus verschiedenen Template-Schnipseln das nötigste zusammenbaut. Dann könnte man z.B. den normalen und den TypeScript-Template-Adapter zusammenführen.

                    1 Reply Last reply Reply Quote 0
                    • Bluefox
                      Bluefox last edited by

                      Bei Google Style Guide kann/muss man auch Ausnahmen machen. 4 Spaces als "indent" ist so eins.

                      1 Reply Last reply Reply Quote 0
                      • M
                        ManfredH last edited by

                        @apollon77:

                        Für die ioBroker-Adapter wäre es super wenn einer das Thema treibt und es einfach macht. `
                        Bin jetzt erst durch https://forum.iobroker.net/viewtopic.php?f=17&t=15654&p=164915#p164915 auf dieses Posting aufmerksam geworden. Und was einen Style-Guide angeht so bin ich gerne bereit, mich hierfür zu engagieren, am besten mit 1-2 anderen, die hier auch schon einschlägige Erfahrung haben.

                        Auf github scheint(!) es irgendwelche Vorgaben zu geben, aber github ist halt eine Entwickler-Platform. Insofern schlage ich vor, die Anwender-Doku ausschließlich auf der ioBroker-Webseite zu veröffentlichen und Installation, Change-Log, Abhängigkeiten, Kurzbeschreibung, technische Details usw. auf github und evtl. parallel dazu auch bei den Fragezeichen hinter den Adaptern. Da gehört das Zeug m.E. nämlich hin. Da ist dann ein Schema dahinter. Und auch für github braucht es m.E. detaillierte Vorgaben. Gutes Beispiel: SQL- vs. Viessmann-Adapter (auf die schnelle, weil ich da grad dran bin). Wobei das github-Readme hinter dem Fragezeichen fast schon Overkill ist. Hinter dem Fragezeichen stelle ich mir eine Kurzbeschreibung vor, Change-Log, Installationsanleitung und vor allem -voraussetzungen (Stichwort "npm" beim Admin-Adapter, mit dem ich eben zu tun hatte). Bei der automatisch generierten Webseite der Adapterliste kann man ja bei github hinter den Links bleiben.

                        Wenn sich außer mir noch ein paar Nasen finden dann sollte man für die einen eigenen Bereich definieren, damit hier nicht allzu viele mitmischen (außer den Entwicklern).

                        1 Reply Last reply Reply Quote 0
                        • M
                          ManfredH last edited by

                          Noch ein Wort zum Style-Guide und wie man das handhabt, daß sich die Leute dran halten. Das kann man m.E. insofern forcieren, als man in der Adapterliste einen Marker einführt für styleguide-konforme UIs, evtl. sogar eine Gruppierung. Wenn ein Adapter dann nicht mehr oben bei den styleguide-konformen Adapter auftaucht, dann besteht durchaus die Möglichkeit, daß sich der eine oder andere Entwickler überlegt, was der tun kann um wieder oben aufzutauchen 😉

                          1 Reply Last reply Reply Quote 0
                          • apollon77
                            apollon77 last edited by

                            Lasst bitte Styleguide und Webseite/Doku nicht zu arg mischen. Zwei Threads - je einen pro Thema - ist ganz ok 🙂

                            Ich schreibe mal im anderen Thread nochwas

                            1 Reply Last reply Reply Quote 0
                            • M
                              ManfredH last edited by

                              Naja - auch für die Webseite (Anwender-Doku) muß (sollte) es einen Styleguide geben! D.h. mehrere verschiedene: * UI

                              • "Fragezeichen"

                              • Anwender-Doku (Webseite)

                              • github

                              1 Reply Last reply Reply Quote 0
                              • First post
                                Last post

                              Support us

                              ioBroker
                              Community Adapters
                              Donate
                              FAQ Cloud / IOT
                              HowTo: Node.js-Update
                              HowTo: Backup/Restore
                              Downloads
                              BLOG

                              572
                              Online

                              32.0k
                              Users

                              80.6k
                              Topics

                              1.3m
                              Posts

                              8
                              24
                              1860
                              Loading More Posts
                              • Oldest to Newest
                              • Newest to Oldest
                              • Most Votes
                              Reply
                              • Reply as topic
                              Log in to reply
                              Community
                              Impressum | Datenschutz-Bestimmungen | Nutzungsbedingungen
                              The ioBroker Community 2014-2023
                              logo