Page MenuHomeWildfire Games

Add a move modifier hotkey
ClosedPublic

Authored by temple on Jun 23 2017, 11:08 PM.

Details

Summary

Fields are different than other buildings in that units can walk on them. The problem is when you click to walk on a field your units will instead try to gather from it (if they can). When you click on an enemy's field you'll either attack it or again try to gather from it. This makes it difficult to maneuver units near fields.
Forests are another example where it would be nice if there was a modifier hotkey that would let you move there rather than gather.
Maybe there could be a situation where you want to move to some enemy structures or units but not attack them.
(With D1074 if you clicked to move to a building, you'd move to the point on the building closest to your current position. With this hotkey instead it would work like it currently does (in a22): you'd move to the nearest point to where you clicked.)

Test Plan

Alt would be my choice of hotkey, but that's now used with the new "order-one-unit-from-selection" functionality, so instead I set it to be unused by default.
To test, set it to your favorite hotkey.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

temple created this revision.Jun 23 2017, 11:08 PM

(Consider the different uses of footprint)

I don't think that it's the good way. It looks strange for me. We have many other resources (animals/trees/stones), we may want to go near of it, but don't gather. Also it'd be harder to click on fields when you're clicking very fast.

So my suggestion is to add a hotkey to avoid any special action (i.e. gather), only movement. I.e. Alt+RMB on a tree only puts an order to go to the tree, but don't gather it.

P.S. I haven't ever met this problem with fields. There weren't needs to place units on fields.

i agree with vladislave because this is "move instead gather" issue occurs while moving in a forest aswell, thus an hotkey would really comes in handy.
By the way grainfields footprint and actor may fit the obstruction size in order to better manage their placement. Ia bigger actor than the real obstruction sizes is tricky whenever a player is trying to dovetail structures in order to optimize space and, f.e. the number of grainfields to place around the civic center.

temple planned changes to this revision.Jun 25 2017, 4:32 PM

I agree, a new hotkey sounds like a much better solution. I'll work on that when I get a chance.

temple updated this revision to Diff 4519.Dec 4 2017, 1:46 AM
temple retitled this revision from Decrease the field's footprint to Add a move modifier hotkey.
temple edited the summary of this revision. (Show Details)
temple edited the test plan for this revision. (Show Details)
temple set the repository for this revision to rP 0 A.D. Public Repository.
temple added a subscriber: wraitii.
Owners added a subscriber: Restricted Owners Package.Dec 4 2017, 1:46 AM
temple edited the summary of this revision. (Show Details)Dec 4 2017, 1:48 AM
Vulcan added a subscriber: Vulcan.Dec 4 2017, 3:08 AM

Successful build - Chance fights ever on the side of the prudent.

Updating workspaces...
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (308 tests)....................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (308 tests)....................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...
Imarok accepted this revision.Dec 5 2017, 9:37 PM

Works and code is good.

This revision is now accepted and ready to land.Dec 5 2017, 9:37 PM
This revision was automatically updated to reflect the committed changes.