Diskussionsissue - Githubissues

rowe42 / lhm_animad_admin_html5

0 stars 6 forks source link

Diskussionsissue #241

Open tderflinger opened 6 years ago

tderflinger commented 6 years ago

Hallo,

wie besprochen hier ein Issue meiner Findings:

Nutzung von let/const anstatt var
Nutzung von .toString anstatt new String()
ES6 Linter strenger einstellen
Bevorzugung forEach / for of / for in anstatt konventionelles for (let x; x < 10;x++)
Funktionsparameter keine Werte zuweisen
Komponenten feingranularer schneiden, nicht zu viel pro Komponente, lieber mehrere Komponenten, auch Kindkomponenten

Grüsse, -Thomas

xdoo commented 6 years ago

@tderflinger bitte in den Code referenzieren. So können wir nicht alle Punkte nachvollziehen.

xdoo commented 6 years ago

@tderflinger

Komponenten feingranularer schneiden, nicht zu viel pro Komponente, lieber mehrere Komponenten, auch Kindkomponenten

Bitte Beispiele benennen und einen Schnitt für diese Beispiele vorschlagen.

rowe42 commented 6 years ago

@tderflinger @a52team Kann hier bitte nochmal jemand reagieren auf unsere Anfrage oben?

tderflinger commented 6 years ago

Zum Beispiel in animad-app.html könnte man noch eine Komponente aus folgenden Zeilen machen:

                        <template is="dom-if" if="{{animalsViewAllowed}}">
                        <a class="paper-item-link" name="animals-view" href="#[[rootPath]]animals-view/" tabindex="-1">
                            <paper-icon-item>
                               <iron-icon icon="animad-icons:animals-view" slot="item-icon"></iron-icon>
                                [[$t('animals')]]
                          </paper-icon-item>
                        </a>
                        </template>

Danach:

` Meiner Meinung sollte man lieber mehrere Komponenten generieren, was die Leserlichkeit erhöht (allerding auch etwas mehr Code).