-
Notifications
You must be signed in to change notification settings - Fork 10
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
Create a JumpPad asset #144
base: master
Are you sure you want to change the base?
Conversation
I am having trouble understanding the reason for this pr? Also, there are some unnecessary changes in your pr. It seems like you opened and committed the game in a newer version of unity. |
You dont have to add it. was just an idea maybe if you create platforming levels
…________________________________
From: Leonel Jara <notifications@github.com>
Sent: Friday, November 16, 2018 2:51:59 PM
To: PopeSpaceous/Solitary
Cc: Oluwaseyi Aketepe; Author
Subject: Re: [PopeSpaceous/Solitary] Create a JumpPad asset (#144)
I am having trouble understanding the reason for this pr?
I don't think we need a jump pad asset.
Also, there are some unnecessary changes in your pr. It seems like you opened and committed the game in a newer version of unity.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#144 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/Ah7O1tm4FXfg91RUDbF9gpDlfHJTcxz3ks5uv0GPgaJpZM4YnD6E>.
|
@saketepe Is this just a png file that has been added or have you wrote a script to go with the jump pad? I do like the Idea of a platform level so it wouldn't be terrible to have a functioning asset. We would have to edit the image to make it more inline with our current style. |
Cool! But i don't know how to write the scripts yet, and how to properly change the style. I have been getting busy so i wont be able to work on anything until later on after exams. But when they are done i should i have to learn that.
…________________________________
From: PopeSpaceous <notifications@github.com>
Sent: Friday, December 7, 2018 5:00:07 PM
To: PopeSpaceous/Solitary
Cc: Oluwaseyi Aketepe; Mention
Subject: Re: [PopeSpaceous/Solitary] Create a JumpPad asset (#144)
@saketepe<https://github.com/saketepe> Is this just a png file that has been added or have you wrote a script to go with the jump pad?
I do like the Idea of a platform level so it wouldn't be terrible to have a functioning asset.
We would have to edit the image to make it more inline with our current style.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#144 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/Ah7O1r8By5WZU0FDDXgF0IkJ4Dhg9oZNks5u2w8XgaJpZM4YnD6E>.
|
@saketepe no rush. Likely won't be merged until after the semester is over. Afterwards I could help you with the styling as I did most of the sprites for the project. Also join the discord channel if you get the chance. @Dragomegak is also learning the scripts for creating assets. |
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.
In order to get this merged we need to setup the Jump Pad into the same style as the rest of the assets.
We also need to create a script for implementing the jump pad into an in game asset.
We also have to clean up the additional files added from using a newer version.
Rn it is just a picture but it should give a bounce effect to the player(Any other files are not to be added unless needed)