Zitat:
Warum speicherst du LastColor, wo du diesen Wert doch bereits in einer anderen Variable kennt?
In welcher Variable kenne ich die zuletzt genutzte Farbe denn außer in FLastLabelColor?
Zitat:
PS: Wenn du während des Hover eine neue Farbe zuweist, dann wird nach dem Leave die falsche Farbe angezeigt.
Das habe ich durch die Hilfe von FLabelHoverColorTmp in SetLabelMouseProperties und abfragen in GetLastLabelColor ausgebessert.
Zitat:
Wozu haben die Set-Methoden einen TLabel-Parameter?
Und warum verwendest du den, anstatt direkt auf Self zuzugreifen?
Habe sie entfernt und greife jetzt auf Self zu.
Zitat:
Bezüglich dem Default-Wert für deine Property solltest du nochmal nachsehen,
denn es macht die
DFM übersichtlicher, wenn Default-Werte (das, was im Constructor gesetzt wurde) nicht sinnlos die
DFM zumüllen.
Das verstehe ich leider nicht so ganz. ich weiß, dass man oben in der Kopfdeklaration auch default so-und-so hinzufügen kann. Aber das wird nicht in den ObjektInspektor übernommen.
Zitat:
Die Color-Property sollten Setter-Methoden bekommen, damit beim Zuweisen auch gleich die Anzeige aktualisiert werden kann.
Das verstehe ich leider auch nicht so ganz.
Zitat:
Und warum eine Magic-Number, anstatt einer ordentlichen Konstante?
Es gibt sogar schon eine Fertige, die man statt der -1 nutzen kann. (Delphi-Referenz durchsuchenclNone / Delphi-Referenz durchsuchenVcl.Graphics.TColor)
Habe ich auch ausgebessert.