Skip to content

Conversation

@Astralcircle
Copy link
Contributor

@Astralcircle Astralcircle commented Nov 24, 2024

Use color_white instead of Color(255, 255, 255, 255)
Use ScrH() / 5 instead of 200
Use less bits for WireLib.AddNotify
Make NOTIFYSOUND_DRIP5 equal NOTIFYSOUND_DRIP4 because this sound does not exist

@Denneisk
Copy link
Member

What's the purpose to changing the location to ScrH() / 5?

@Astralcircle
Copy link
Contributor Author

Astralcircle commented Nov 24, 2024

What's the purpose to changing the location to ScrH() / 5?

For supporting different resolutions, isn't it logical? The text position shifts only slightly.

@Denneisk
Copy link
Member

Huh, seems like it's some developer leftover that isn't even used anywhere. I thought it was related to the Wire debugger and was worried making the position dynamic might mess with how it avoids the toolgun HUD.

@Astralcircle
Copy link
Contributor Author

Astralcircle commented Nov 25, 2024

I think this dead code should not be removed, as some addons that use WireLib may rely on it

@Denneisk Denneisk merged commit 95311d6 into wiremod:master Nov 26, 2024
1 check failed
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