Forgetting to set a PopupMenu causes runtime exception

bgrabitmap / bgracontrols

🆗 BGRA Controls is a set of graphical UI elements that you can use with Lazarus LCL applications.

https://bgrabitmap.github.io/bgracontrols/

189 stars 32 forks source link

Closed sganz closed 6 months ago

sganz commented 7 months ago

Submitted a small pull request.

In ColorSpeedButton.pas:560

  if PopupMode then
  begin
    p := Parent.ClientToScreen(Point(Left, Top));
    PopupMenu.PopUp(p.x, p.y + Height);
  end;

If you forget to set the PopupMenu property after settintg the PopupMode to True, you can have a runtime exception.

This is in this Pull Request

lainz commented 6 months ago

Applied pull request. Thanks.