Warum SetCurrentDir und nicht
FindFirst(path+'\skins\'+'*',...
?
Warum heißt die Variable Path, obwohl da ein Directory drin ist?
Wann wird Files aufgerufen?
Das Item (eigentlich Sender) sollte zwar das Menüitem sein, aber sicherheitshalber kann
(Item as TMenuItem).Enabled:=false;
nicht schaden. (vorallem wenn man nicht ganz sicher ist, was man da macht)
Und vorallem, wer kommt denn bitte auf die saublöde Idee und nennt eine Methode OnClick?
Ganz besonders dann, wenn die TForm1 bereits ein Property mit dem selben Namen besitzt.
Ach ja, Enabled blendet nichts aus.
(Visible)