Skip to content
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

D2k map pool update - november 2023 #21221

Merged
merged 1 commit into from
May 4, 2024

Conversation

Porenutak
Copy link
Contributor

First commit adds new cliff passages that these (and manny other) maps use. All new tiles was created by me and are free to use.
Second commit update map pool:
new maps: worm_graveyard.oramap
update maps: red-chasm.oramap, death-depths.oramap, the-duell.oramap
removed maps: cliffmaze

Updated maps changelog: equal distance from spice fields for each player, equal amount of spice for each player and minor balance adjustments.

Ps. Worm graveyard use repairble passages by enginear.

@PunkPun

This comment was marked as outdated.

@Porenutak

This comment was marked as outdated.

pchote

This comment was marked as outdated.

@Porenutak Porenutak force-pushed the Map-pool-update-November-2023 branch from 6751d47 to 0f176d9 Compare December 3, 2023 16:12
@Porenutak

This comment was marked as outdated.

@Porenutak Porenutak force-pushed the Map-pool-update-November-2023 branch from 0f176d9 to 512e28d Compare December 3, 2023 16:16
Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure how to feel about this feature. It's very hidden away so likely only spreadable by mouth to mouth. It also feels unpolished, should tooltips be in lower case?

Passage2 Passage1
Capture.mp4

@Porenutak
Copy link
Contributor Author

This feature is here -/+ year and a half. Its part of Enhanced maps and other map mods inherit it to. The reason why engi didnt dissapier is because of balance. Noone wanna spend 400$ just for cliff repair that can be destroyed again.
Ussually this feautre is used as obsticle to capture neutral building in early game.
Snímka obrazovky (26)
Snímka obrazovky (25)

If you feel its a problem a can talk to map author if he agree to remove them.
My English is terrible, so if someone come with better tooltip description I will include it.

@Porenutak
Copy link
Contributor Author

I'm not sure how to feel about this feature. It's very hidden away so likely only spreadable by mouth to mouth. It also feels unpolished, should tooltips be in lower case?>

Maybe I should add Briefing description?

@Porenutak Porenutak force-pushed the Map-pool-update-November-2023 branch from 512e28d to 7040a63 Compare December 6, 2023 22:51
@PunkPun
Copy link
Member

PunkPun commented Dec 15, 2023

I'm not sure what's the best way.

Perhaps the fist step would be not requiring force attack to kill it? It just feels like the cliff is basically invisible and one has no way to learn about it unless explicitly taught

@Porenutak
Copy link
Contributor Author

This feature should act same as for example bridges in RA. You dont wanna autotarget brige, you wanna past thru it or stay there or destroy it only when situation require that (same as in RA). So force force is the good way. Repaireble cliffs are shown on Radar as actors and tooltip give you more info. I dont know how to hint players more without explicit description in mission briefing (which I do when this feautre come out)

@abcdefg30 abcdefg30 added this to the Next Release milestone Dec 16, 2023
@Porenutak Porenutak force-pushed the Map-pool-update-November-2023 branch 2 times, most recently from 2f491e0 to f9bcbe5 Compare January 8, 2024 11:06
@Porenutak
Copy link
Contributor Author

Rebased.

Worm cementery map:

  • custom tiles converted from r8 - r16
  • add description to engineer, that he can repair destroyed cliffs
  • changed destroyed cliffs into repaired state, so even if someone didnt notice this map feature it wouldnt be a problem.

@Porenutak
Copy link
Contributor Author

Map rules use translation now.

@PunkPun
Copy link
Member

PunkPun commented Jan 21, 2024

The discussion on discord indicates that there's a desire to have tiles defined at a game level instead of by the map

@Porenutak
Copy link
Contributor Author

Porenutak commented Jan 22, 2024

  • Destroyable tiles now defined inside the mod. Assets moved into bits folder
  • engineer description updated
  • fixed AI attacking cliffs

@Porenutak
Copy link
Contributor Author

removed some redundant lines in destroyable-tiles.yaml

mods/d2k/languages/rules/en.ftl Outdated Show resolved Hide resolved
mods/d2k/languages/rules/en.ftl Outdated Show resolved Hide resolved
PunkPun
PunkPun previously approved these changes Mar 3, 2024
Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

- add Destroyable tiles into d2k mod
- add ability engineer to repair destroyed cliffs
@Mailaender Mailaender force-pushed the Map-pool-update-November-2023 branch from 371ddcd to b66609d Compare May 4, 2024 14:48
@Mailaender Mailaender merged commit 8c17455 into OpenRA:bleed May 4, 2024
3 checks passed
@Mailaender
Copy link
Member

Changelog

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

Successfully merging this pull request may close these issues.

None yet

5 participants