-
Notifications
You must be signed in to change notification settings - Fork 34
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Closes #5 by adding ability to specify view
Added ability to specify view for facial avatar; front, left, right, or back
- Loading branch information
1 parent
8d77e5e
commit 415ca14
Showing
3 changed files
with
76 additions
and
16 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
415ca14
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanx but i mean this for (https://github.com/jamiebicknell/Minecraft-Avatar/blob/master/skin.php) because it is now every rotate at one.
415ca14
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I seemed to have misunderstood your original idea, sorry. I do think this is a good addition though and I'm glad I built in this feature, so thanks for that anyway.
Regarding the skin.php, it will cause some issues because you specify the width (and the height is automatically calculated). So if you have two images; front and left side, each with a size (width) of 200, then the left size will be much taller.
Another issue is that the nature of the preview is to preview all sides, it was built for this purpose alone, and that's why it has padding horizontally and vertically so that it fits nicely in a fixed with area.
What you are suggesting will require a new feature that doesn't have the padding, and only shows one whole view, and that the size sets the height so that the height remains constant for different views. I feel this overcomplicates what I'm going for with this script, but will give it some thought anyway.
415ca14
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this really would work it would be very nice. For me it still in no hurry since I no pictures use now, but I do want to do this.
Thank you very much if you made this !!
Sorry for my bad English i'm Dutch.