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

Add roadsignstext special world property #3385

Merged
merged 3 commits into from
May 23, 2024

Conversation

FileEX
Copy link
Contributor

@FileEX FileEX commented May 12, 2024

Added roadsignstext special world property, which allows you to disable text rendering on road signs. Thanks to this, we have the opportunity to make our own road signs, which was suggested e.g. in #672

Enabled:
image

Disabled:
image

@samr46
Copy link
Contributor

samr46 commented May 17, 2024

Just wanna let you know again that current implementation is not backwards compatible (my comment with details in PR #3365)

@FileEX
Copy link
Contributor Author

FileEX commented May 17, 2024

Just wanna let you know again that current implementation is not backwards compatible (my comment with details in PR #3365)

I don't see your comment in that PR

@samr46
Copy link
Contributor

samr46 commented May 17, 2024

I don't see your comment in that PR

You need to create new struct for this property and add new eBitStreamVersion in order for it to be backwards compatible. Please look at how "fireballdestruct" is implemented. There are also references to this struct in CMapInfoPacket.cpp and CPacketHandler.cpp

Copy link
Member

@Lpsd Lpsd left a comment

Choose a reason for hiding this comment

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

Looks good to me

@Lpsd
Copy link
Member

Lpsd commented May 23, 2024

Please update https://wiki.multitheftauto.com/wiki/SetWorldSpecialPropertyEnabled once the next nightly build has been created (after r22420).

@Lpsd Lpsd merged commit 4a746ec into multitheftauto:master May 23, 2024
6 checks passed
MTABot pushed a commit that referenced this pull request May 23, 2024
4a746ec Add roadsignstext special world property (#3385)
@FileEX FileEX deleted the feature/roadsign_property branch May 23, 2024 19:16
@TheNormalnij TheNormalnij added this to the 1.6.1 milestone May 25, 2024
@TheNormalnij TheNormalnij added the enhancement New feature or request label May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants