Skip to content
GitLab
Closed
Issue created by Neil Brown@nccbMaintainer

[GREENFOOT-634] Convert set-image dialog

The set-image dialog currently lives in greenfoot.gui.images (move to .guifx. as you convert). Bear in mind that this dialog has twin uses: setting the image for an existing class, and creating a new class with a specified image.

With hindsight, I'm not totally sold on the cog menu in this dialog, so I think there's some room for improvement. One change we could make is that we don't currently have a right-click menu for images. The Edit, Duplicate and Delete items on the cog menu could be moved to a context menu, and the Create, Paste and Import could perhaps go back to being buttons. (Or we could put a name for the menu rather than a cog?)

Don't forget to swap uses of GClass to ClassTarget.

Update: there is now a GreenfootStage.setImageFor method which is wired up to the context menu. So the invocation of the converted set-image dialog should go in that method.

Issue metadata

  • Issue type: Task
  • Priority: Medium