New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When changing rally point also make primary #12855 #12977

Merged
merged 1 commit into from Apr 11, 2017

Conversation

Projects
None yet
5 participants
@rob-v
Contributor

rob-v commented Mar 17, 2017

Set PrimaryBuilding together with RallyPoint when Ctrl is pressed for each building which is the only selected building of that type. Ctrl is optional, could be removed to set PrimaryBuilding.

Barracks + Light factory, Ctrl+Right click = RallyPoint + PrimaryBuilding

rallypointwithprimarybuilding

2 Barracks = only RallyPoint

rob-v added a commit to rob-v/OpenRA that referenced this pull request Mar 17, 2017

@abcdefg30

Looks good to me. 👍 /

@rob-v

This comment has been minimized.

Show comment
Hide comment
@rob-v

rob-v Mar 20, 2017

Contributor

ExtraDataSetPrimaryBuilding renamed to ForceSet and code applying ForceSet moved from RallyPoint to PrimaryBuilding.

Contributor

rob-v commented Mar 20, 2017

ExtraDataSetPrimaryBuilding renamed to ForceSet and code applying ForceSet moved from RallyPoint to PrimaryBuilding.

@rob-v

This comment has been minimized.

Show comment
Hide comment
@rob-v

rob-v Mar 22, 2017

Contributor

@atlimit8 Thanks for you comments and proposals. Updated.

Contributor

rob-v commented Mar 22, 2017

@atlimit8 Thanks for you comments and proposals. Updated.

@abcdefg30

This comment has been minimized.

Show comment
Hide comment
@abcdefg30

abcdefg30 Apr 11, 2017

Member

My 👍 is still valid.

Member

abcdefg30 commented Apr 11, 2017

My 👍 is still valid.

@atlimit8 atlimit8 merged commit 6b18eb1 into OpenRA:bleed Apr 11, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@atlimit8

This comment has been minimized.

Show comment
Hide comment
@atlimit8
Member

atlimit8 commented Apr 11, 2017

@rob-v rob-v deleted the rob-v:RallyPointPrimaryBuilding branch Apr 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment