PlayerSkin.js doesn't support helm #91


  • Enhancment
  • Accepted
Open
Assigned to snarfattack
  • TNTUP created this issue May 11, 2013

    What steps will reproduce the problem?
    1. Check a player statistics, the canvas body (where the player skin loads) doesn't load helm layer (If it has one)
    2. My skin has an helmet layer
    3.

    What is the expected output? What do you see instead?

    Seeing the player's helmet

    What version of the product are you using?
    v2.18 for 1.5.1

    Please provide any additional information below.

    If its possible, thanks. Also player heads in the player list are blurred... for that size it would be better 16x16, not 8x8 being resized.. goes blur (Doesn't show helmet too)

    I "could" tweak the main page so it will load player faces (16x16) from my webserver (Minotar ones) they're generated from my webserver.

    Or but this is optional, instead of saving player's skins from local disk, they would load from my webserver or minotar's server. You decide :P

  • TNTUP added the tags New Enhancment May 11, 2013
  • TNTUP posted a comment May 11, 2013

    Edit: Successfully changed player list heads, currently without changes, the player's face looks blurred... 8x8 image I think... Changed values to "img" tag then put player's helm face (16x16) in, looks better :3

    test Instead of blurred faces t

    You notice the 1st image, players faces are much nicer and they have their helm layer. The second one, players faces are blurred and they dont have their helm layer.

    I can tell the edits I did, I could try editing playerskin.js and checking if i can get the helm working. Faces are being loaded from my webserver, always working 24/7

    And... uhm I suck at js so I dont think im able to edit it... unlike java applets they show player's helm.. could add helmet layer if its possible?


    Edited May 11, 2013
  • TNTUP edited title May 11, 2013
  • TNTUP posted a comment May 13, 2013

    NVM PlayerSkin.js ALREADY has helmet fuction inside, but it was // commented. Removed it and it worked <3

    Why you did "//" for the helment fuction? Some minecrafters has helms... Well I've fixed it myself. Though you can fix the blur faces in teh player list? I've edited it to load faces from my webserver.


    Edited May 13, 2013
  • snarfattack unassigned issue from _ForgeUser7609849 Feb 8, 2014
  • snarfattack self-assigned this issue Feb 8, 2014
  • snarfattack removed a tag New Feb 8, 2014
  • snarfattack added a tag Accepted Feb 8, 2014

To post a comment, please login or register a new account.