Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- [[ 7ad7139 ]] Armin:
- - Irgendwie hast du es geschafft die ProjektID in Android.csproj zu ändern - keine Ahnung wie, aber wär nett wenn du beim comitten schauen könntest was du da pushed
- - Ich habe keine Ahnung was ich mit Klassen im Testing namespace machen soll, ich würde sagen erstell richtige Objekte oder commite die nicht
- - Deine sourcedateien sind jetzt ein misch-masch aus tabs und spaces - das ist irgendwie noch schlimmer als vorher :D
- - Die xaml Datei ist voller Warnings, ich glaube das android projekt hat bewiesen dass niemand im nachhinein durchgeht und die später fixed
- - ios:Page.UseSafeArea sollte man nur im top level element verwenden und nicht mitten in der hierachie, wie in einer tabpage
- - bei den anderen seiten haben wir einen expliziten margin, von mir aus können wir auch ios:Page.UseSafeArea benutzen - nur eine einheitliche Lösung wäre schön
- - die xaml dateien bitte nicht so wie android formatieren und (grundlos) alle attribute in extra zeilen machen
- - allgemein die xaml datei auch ordentlich formatten, mit einheitlicher enrückung und keinen random leerzeilen (wenn sie zur überischtlichkeit beitragen sind sie ok)
- - x:Name sollte allgemein fast nie verwendet werden weil der datenfluss über bindings kommen/gehen sollte
- - Kein Code in die Page Klasse stecken - maximal redirects für die click events (und da schaue ich mal wenn ich zeit habe dass wir das auch binden können)
- - Der "as" Parameter in C# ist ein conditional-cast wenn du den sowieso nicht überprüfst dann mach ein hard-cast - ist schneller und im problemfall sagt die exception wenigstens was aus
- - Methoden mit default (internal) Sichtbarkeit sind außer in Ausnahmefällen falsch. (das sind auch alles Warnings)
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement