fire-eggs / CivOne

An open source implementation of Sid Meier's Civilization 1 using C# and .NET.
Creative Commons Zero v1.0 Universal
21 stars 4 forks source link

Clicking in view map interrupts Goto destination selection #12

Closed fire-eggs closed 2 years ago

fire-eggs commented 5 years ago

In Microprose, you can click on the inner map to move the viewpoint during a Goto destination selection. This way the destination is visible.

In current CivOne, the Goto is interrupted by clicking in the inner map.

fire-eggs commented 5 years ago

The implementation in Goto.cs "just seems wrong". Goto.cs is a screen, and ignores/escapes when the user clicks in the minimap. Since it is a screen, it doesn't seem to be possible to change the game view (GamePlay.X, GamePlay.Y).

I believe the "correct" implementation is a "mode" in GamePlay.cs. When in 'Goto' mode, switch the cursor to 'Goto', and allow minimap click to move the GameView.

iegik commented 2 years ago

I've implemented "just work" case for minimap