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 embedded PNG texture modifier #11498

Merged
merged 2 commits into from Oct 13, 2021
Merged

Conversation

hecktest
Copy link
Contributor

This PR builds upon #11485 by adding a texture modifier called [png: which allows you to send ephemeral textures to clients in the form of base64 PNG binaries. You can use it for captchas, signs and all sorts of other procedural fun.

mandels

How to test

Follow the instructions in doc/lua_api.txt, or check out the new devtest nodes:

testnodes:generated_png_emb
testnodes:generated_png_src_emb
testnodes:generated_png_dst_emb

Ready for review.

@celeron55
Copy link
Member

Looks good

@hecktest
Copy link
Contributor Author

@celeron55 Then approve :o

@hecktest
Copy link
Contributor Author

Reauthored correctly as requested.

@Emojigit
Copy link
Contributor

Emojigit commented Aug 1, 2021

Looks good! afaik that allows custom player texture since this does not require new images in the filesystem, since Minetest disallows adding extra media during client runtime. Ping @Wuzzy2 , he is the MCL2 main contributor, I think this can resolve the custom skin problem of MCL2.

@Wuzzy2
Copy link
Contributor

Wuzzy2 commented Aug 1, 2021

I'm no longer active in MCL2 and other developers took over but nice try. :P

@Emojigit
Copy link
Contributor

Will this requires client-side changes? If yes, how to get the client's version?

@rubenwardy rubenwardy added the Concept approved Approved by a core dev: PRs welcomed! label Aug 14, 2021
doc/lua_api.txt Outdated Show resolved Hide resolved
Copy link
Member

@sfan5 sfan5 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, CI tests this already.

please look at the review comment though

@kilbith
Copy link
Contributor

kilbith commented Sep 1, 2021

5.5 milestone please.

@Emojigit
Copy link
Contributor

Emojigit commented Sep 2, 2021 via email

@NathanSalapat
Copy link
Contributor

I would expect the client to show an unknown texture error and use a random color. Much as happens now it you forget to include a texture, or have a typo in the texture name.

@appgurueu
Copy link
Contributor

I would expect the client to show an unknown texture error and use a random color. Much as happens now it you forget to include a texture, or have a typo in the texture name.

Plus a "generateImage()" error, "unknown texture modifier" or something.

@sfan5
Copy link
Member

sfan5 commented Oct 6, 2021

I will apply the suggested doc change and merge this soon for 5.5 unless anyone has any complaints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Client / Audiovisuals Concept approved Approved by a core dev: PRs welcomed! Feature ✨ PRs that add or enhance a feature One approval ✅ ◻️ Textures
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants