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 sample texture with cape #1393

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
6 participants
@Rogier-5
Copy link
Contributor

commented Nov 17, 2016

I propose to add an example texture with cape.

screenshot_20161117_104149

NOTE: this will not be the default texture. Just an additional png file that is included as an example of how to add a cape to the model.

@DS-Minetest

This comment has been minimized.

Copy link
Contributor

commented Nov 17, 2016

i dont think, this has to be reported in minetest game

@sofar

This comment has been minimized.

Copy link
Member

commented Nov 17, 2016

I'm ok with adding this template, and adding it here.

But please use _ and not - in file names.

@paramat paramat added the Textures label Nov 18, 2016

@C1ffisme

This comment has been minimized.

Copy link

commented Nov 18, 2016

Paramat talked about removing the cape altogether, but I don't remember what happened to that conversation.

@Rogier-5 Rogier-5 force-pushed the Rogier-5:player-with-cape branch from 2ee4dda to 5d43bbb Nov 18, 2016

@Rogier-5

This comment has been minimized.

Copy link
Contributor Author

commented Nov 18, 2016

@sofar: I should have realized that... Fixed.

@DS-Minetest

This comment has been minimized.

Copy link
Contributor

commented Nov 18, 2016

with this some players will just take the texture and replace it

maybe it would be good to

  • make the cape to the texture by default
  • or make a setting to enable capes

but i dont see any use for an example there...

@Rogier-5

This comment has been minimized.

Copy link
Contributor Author

commented Nov 18, 2016

@DS-Minetest I think that many people will prefer a character without cape. So I think it's better to not have a cape by default. However I personally wouldn't mind if the default character had a cape.

Having a cape or not is just a matter of choosing the right texture. See this mod. I don't know if it would be possible to actually disable just the cape. Or an arm or leg for that matter. Anyway, if it's doable at all, then that would be a minetest-core change.

@paramat

This comment has been minimized.

Copy link
Member

commented Nov 18, 2016

make the cape to the texture by default
or make a setting to enable capes

I disapprove of both,

Since this PR is for adding an example of how to add a cape to the player skin, if something like this is added perhaps better to include a large template with all texture sections labelled, like this plus cape:

skinzones

This would be best added in the mod folder, not in 'textures'.
However i'm still unsure whether something like this belongs here.

My general objection to capes is because they are a copy of something very Minecrafty, capes in Minecraft are very much associated with Minecon exclusives, so capes seem a little like distasteful copying, like Minecraft adding Mese block.

@Rogier-5 Rogier-5 force-pushed the Rogier-5:player-with-cape branch from 5d43bbb to 0e81c22 Nov 18, 2016

@DS-Minetest

This comment has been minimized.

Copy link
Contributor

commented Nov 19, 2016

i agree @paramat's opinion to make a template like this
but capes arent really typical for minecraft. thats not comparable with mese

@rubenwardy

This comment has been minimized.

Copy link
Member

commented Nov 22, 2016

Wouldn't this be better on the wiki?

@paramat

This comment has been minimized.

Copy link
Member

commented Nov 23, 2016

Yes i doubt the mod is the right place for templates 👎

@paramat

This comment has been minimized.

Copy link
Member

commented Nov 28, 2016

ShadowNinja's opinion http://irc.minetest.net/minetest-dev/2016-11-28#i_4751640
Closing due to 3 -1s.

@paramat paramat closed this Nov 28, 2016

@paramat paramat added the Won't add label Nov 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.