Decorators

[Paulomart]

Wir haben aktuell keinen Standart wie wir Decorators benutzen.

@5ebastianMeier und ich haben heute bereits darüber diskutiert. Grob gibt es zwei verschiedene Vorschläge:

Gleiche Zeile:

class Fuu {
  @bindable private member;
}

Neue Zeile:

class Bar {
  @bindable
  private member;
}

---

Außerdem können Decorators auch an Klassen erscheinen:

Gleiche Zeile:

@inject class Baz {
}

Neue Zeile:

@inject
class Fuu {
}

---

Leider gibt es aktuell keine tslint oder eslint Konfiguration dazu.

Das Issue betrifft TypeScript genauso wie JavaScript

[Paulomart]

Meiner Meinung nach gehören die Decorator in eine neue Zeile, falls mehrere Decorators an einem Feld oder einer Klasse sind, bringt das mehr Übersicht.

[heikomat]

@Paulomart ist ein argument. Allerdings sieht das auch nicht schön aus, wenn man z.b. 3 einfache bindables hat

[Paulomart]

@heikomat

Ich wäre trotzdem dafür wenn wir uns für eine Sache entscheiden das einheitlich durchzuziehen: Bei Klassen, Funktionen und Feldern.

Für mich sieht das hier einfach unschön aus:

@inject('Fuu', 'Bar', 'Baz') @wowDecorator('such awesome') export class Zap {
  ...
}

Dagegen ist dies viel leserlicher:

@inject('Fuu', 'Bar', 'Baz')
@wowDecorator('such awesome')
export class Zap {
  ...
}