238: save viewMode of presenceControl

[spahrson]

#238

[caebr]

Sieht gut aus, manuelle Tests i.O.

Noch eine Frage - in der Story sehe ich:
Key "presenceControlListOrPhoto": (true/false)

In der Umsetzung ist es aber ein String ("grid", "list"). Was aber eigentlich auch mehr Sinn macht...

[spahrson]

In der Umsetzung ist es aber ein String ("grid", "list"). Was aber eigentlich auch mehr Sinn macht...

Ja ich habe die Werte des integrierten ViewMode Enums übernommen. Somit ist das flexibel (falls mal ein neuer oder anderer ViewMode hinzukommen sollte. Aber stimmt, ich habe das nicht mit Sandro besprochen. @schefbi ist das ein Problem oder ist es ok, wenn ich flexibel den Wert des ViewModes (aktuell ein Wert aus ['grid' | 'list']) anstelle fix true/false abspeichere? Möchtest du das Propery noch namentlich anpassen?

[schefbi]

In der Umsetzung ist es aber ein String ("grid", "list"). Was aber eigentlich auch mehr Sinn macht...

Ja ich habe die Werte des integrierten ViewMode Enums übernommen. Somit ist das flexibel (falls mal ein neuer oder anderer ViewMode hinzukommen sollte. Aber stimmt, ich habe das nicht mit Sandro besprochen. @schefbi ist das ein Problem oder ist es ok, wenn ich flexibel den Wert des ViewModes (aktuell ein Wert aus ['grid' | 'list']) anstelle fix true/false abspeichere? Möchtest du das Propery noch namentlich anpassen?

Finde ich gut nennt doch das Property presenceControlViewMode.