godot-port icon indicating copy to clipboard operation
godot-port copied to clipboard

Enhance rotate and zoom UI

Open CrafterSvK opened this issue 1 year ago • 7 comments

Hello, I've been trying to get a hold of the codebase, so I've chosen the simplest thing I could think of.

  • [x] Zooming in and out using buttons and mouse
  • [x] Rotation
  • [x] Zoom buttons should be disabled on min/max zoom

Rotation is currently handled by multiple keyboard input handlers which do separately things (rotate camera, rotate isometric objects). Need to merge to one keyboard handler which sends signals to other handlers. In my opinion it would be lazy to solve this by making a global emitter, for these kinds of things, maybe that's right, I do not know. Or just make button to click rotate_left, rotate_right key :)

CrafterSvK avatar Aug 06 '23 15:08 CrafterSvK

Every WorldThing can now react to rotation of camera. Maybe the emit to Global camera rotation from buttons can be directly made in TabWidget so there will be no need to signal it through PlayerHUD to CameraControls.

CrafterSvK avatar Aug 07 '23 00:08 CrafterSvK

Buttons are now disabled on max/min zoom. It's handled using Global singleton, maybe refactoring to different Singleton would be better. Or if you find another way to do this, I would be glad.

CrafterSvK avatar Aug 13 '23 18:08 CrafterSvK

@artism90 check it out if you've got time

CrafterSvK avatar Aug 20 '23 09:08 CrafterSvK

I addressed the issues, thanks

CrafterSvK avatar Sep 04 '23 16:09 CrafterSvK

@CrafterSvK have you maybe time to get this pr done please?

LinuxDonald avatar Dec 11 '23 17:12 LinuxDonald

@CrafterSvK how it looks on this pr?

LinuxDonald avatar Jul 16 '24 21:07 LinuxDonald

Hello, I can take a look this evening. This probably will need a rebase first.

CrafterSvK avatar Jul 17 '24 09:07 CrafterSvK