Skip to content

Remove PARENT_TOP and PARENT_LEFT from integer expressions for widgets#21605

Merged
PunkPun merged 1 commit intoOpenRA:bleedfrom
dragunoff:feature/remove-parent-top-parent-left
Oct 3, 2024
Merged

Remove PARENT_TOP and PARENT_LEFT from integer expressions for widgets#21605
PunkPun merged 1 commit intoOpenRA:bleedfrom
dragunoff:feature/remove-parent-top-parent-left

Conversation

@dragunoff
Copy link
Copy Markdown
Contributor

@dragunoff dragunoff commented Oct 3, 2024

PARENT_TOP and PARENT_LEFT should be 0 so they are not very useful substitutions. They are replaced with 0 or removed if that was the whole value for a field.

@dragunoff dragunoff changed the title Remove PARENT_TOP and PARENT_LEFT from integer expressions for widgets Remove PARENT_TOP and PARENT_LEFT from integer expressions for widgets Oct 3, 2024
@dragunoff dragunoff changed the title Remove PARENT_TOP and PARENT_LEFT from integer expressions for widgets Remove PARENT_TOP and PARENT_LEFT from integer expressions for widgets Oct 3, 2024
PARENT_TOP and PARENT_LEFT should be 0 so they are not very useful substitutions. They are replaced with 0 or removed if that was the whole value for a field.
@dragunoff dragunoff force-pushed the feature/remove-parent-top-parent-left branch from 6bcbae7 to 2f15678 Compare October 3, 2024 15:06
@dragunoff
Copy link
Copy Markdown
Contributor Author

Update: reference the update rule in UpdatePath.cs

@PunkPun PunkPun merged commit a69ea79 into OpenRA:bleed Oct 3, 2024
@PunkPun
Copy link
Copy Markdown
Member

PunkPun commented Oct 3, 2024

changelog

@dragunoff dragunoff deleted the feature/remove-parent-top-parent-left branch October 3, 2024 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants