glest / glest-source

Glest is a network multi-player cross-platform 3D real-time strategy (RTS) game, where you create armies of units and battle different factions.
https://glest.io
GNU General Public License v3.0
58 stars 40 forks source link

Implement double right-click to move #167

Closed andy5995 closed 5 years ago

andy5995 commented 6 years ago

A desired feature is to implement double right-click to move units instead of attacking. We tried to implement this feature, but unfortunately it was found to be inconsistent, and not work well during network games.

andy5995 commented 6 years ago

I couldn't either, but it's happened to me a couple times and to @biels so let's keep this ticket open for now.

biels commented 6 years ago

Is it possible that the feature exists on the latest develop but when playing we're using the "release" version and we still don't have it?

andy5995 commented 6 years ago

That's right.

biels commented 6 years ago

It is working on the latest version that I finally got to compile.

biels commented 6 years ago

However I think the binding should be either configurable or hardcoded but switched up in that 1 click -> move, 2 clicks -> attack. Should i open a separate issue?

andy5995 commented 6 years ago

Playing with @Jammyjamjamman earlier tonight, I had this problem several times. Still not sure how to reproduce it every time.

However I think the binding should be either configurable or hardcoded but switched up in that 1 click -> move, 2 clicks -> attack. Should i open a separate issue?

Yes, please do, @biels . I think having it configurable would be a good option.

andy5995 commented 6 years ago

I forget about the double-right click. I'll tell @alketii and remember to try it some more.

biels commented 6 years ago

I'd make it configurable if possible. If not I would reverse it since I think that it is more appropriate and would solve the problem.

On Tue, Sep 25, 2018, 21:47 Andy Alt notifications@github.com wrote:

I forget about it. I'll tell @alketii https://github.com/alketii and remember to try it some more.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ZetaGlest/zetaglest-source/issues/167#issuecomment-424477303, or mute the thread https://github.com/notifications/unsubscribe-auth/AE64MOPGGL7g15w4_gm34LxVO1y0-qIyks5ueohigaJpZM4U0Ep4 .

linux-player9 commented 5 years ago

The double click to move was a good feature since unit attack by default