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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skateboard content expansion pack #64269

Merged
merged 15 commits into from
Feb 8, 2022
Merged

Skateboard content expansion pack #64269

merged 15 commits into from
Feb 8, 2022

Conversation

itseasytosee
Copy link
Contributor

@itseasytosee itseasytosee commented Jan 20, 2022

About The Pull Request

(Drafted because working on balance and looking for more rad ideas)

Makes skateboarding a hell of a lot cooler,

Lowers stamina cost for ollieing and grinding.
Makes collisions with other carbons always result in a wipeout no matter your stamina damage, colliding with a carbon will stun and knock them down for a short time (Stunned for the quarter of the length the skater is, and knockdown for have the length) The time is doubled if you collide with them while they are performing a sick grind.

Additionally grinding on top of stuff now has more functionality.
Grinding over a person that is laying down will deal significant damage with a high chance to wound at the cost of the skaters' stamina. For this to happen the person has to be laying down on the table, if they are standing they will simply collide as normal.
You can grind into a disposals bin.
Fixed action buttons always being red
fixed being able to ollie while unconscious/dead
Grinding over an object that is cuttable with a knife will lead to that item getting instantly cut! This includes cheese.

  • Implement further suggestions for grind functionality
  • Fix some known skateboard bugs
  • Tweak some stamina cost numbers, maybe stun times
  • Make a better changelog
  • Record a video where my character doesn't randomly turn invisible because of a memory shortage.

Why It's Good For The Game

jackass.mp4
dumped.mp4
bonk.mp4

Changelog

馃啈 itseasytosee
expansion: Skateboards have been made considerably more radical
expansion: While grinding on a table with a skateboard, if you grind on top of somebody you are likely to hurt them quite badly
expansion: Colliding with another person will now always result in a crash, in which the person hit gets knocked over as well as the skater (protip: crash into someone while grind to double the stun time!)
expansion: Colliding with a disposal unit while grinding will result in you getting dunked in! Wack!
fix: fixed some skateboard related bugs, including the action button for ollieing always being red, and being able to ollie while dead
/:cl:

@ArcaneMusic
Copy link
Contributor

ArcaneMusic commented Jan 20, 2022

"Who's ready to cut the cake?"
SPACE TONY HAWK GRINDS IN, OLLIES THROUGH THE AIR KNOCKING A CHILD DOWN, CRYING
"WooooOOOOOOOOOAH!"
THE HAWKSTER TURNS THE CAKE INTO 8 SLICES OF PERFECTLY CUT CONFECTIONARY, CHILDREN FLYING LEFT AND RIGHT TO DODGE THE ELDERLY SKATER'S REDICULOUS GRINDS
"DUUUUUUUUUUUUUDE"
THE LIVING LEGEND 360 NOSEGRINDS OUT AN AIRLOCK

@timothymtorres
Copy link
Member

But can you grind inside a disposals half-pipe?

@EricZilla
Copy link
Contributor

EricZilla commented Jan 20, 2022

Always wished there was a positive skater quirk that would shorten your stuntime when you collide with walls (or even just airlocks)

Also you should make it so turning on a jetpack while on a skateboard makes it take less stamina lmao

@itseasytosee
Copy link
Contributor Author

Added grinding into disposals

dumped.mp4

@itseasytosee
Copy link
Contributor Author

Fixed action buttons always being red, fixed being able to ollie while unconscious/dead

bonk.mp4

@Bawhoppen
Copy link
Contributor

Good PR, PKP would be proud 馃憤

@itseasytosee itseasytosee marked this pull request as ready for review January 21, 2022 01:22
@itseasytosee
Copy link
Contributor Author

trade offer

@BadIsMyUsername
Copy link

Added grinding into disposals

Can you grind OUT of disposals? Like if someone placed a table at the exit chute, would you continue on the epic grind? Imagine the skate parks possible if so...

@Likteer
Copy link
Contributor

Likteer commented Jan 23, 2022

@itseasytosee

beauty.mp4

@itseasytosee
Copy link
Contributor Author

@itseasytosee

Thank you, it was inspired

@itseasytosee
Copy link
Contributor Author

This should be ready to go for maintainer review, I haven't collected any more ideas on par with the ability to cut cake by skateboarding over it. But I might come back to it and open another PR with more stuff.

as of now :shipit:

@tohg7
Copy link
Contributor

tohg7 commented Jan 23, 2022

This should be ready to go for maintainer review, I haven't collected any more ideas on par with the ability to cut cake by skateboarding over it. But I might come back to it and open another PR with more stuff.

as of now :shipit:

One thing you could add is an expensive "Pro Skater" quirk that allows you to spawn with a skateboard in your bag and have reduced self-stun times when crashing into someone + increased stun times for the person you crashed into.

@itseasytosee
Copy link
Contributor Author

itseasytosee commented Jan 23, 2022

This should be ready to go for maintainer review, I haven't collected any more ideas on par with the ability to cut cake by skateboarding over it. But I might come back to it and open another PR with more stuff.
as of now :shipit:

One thing you could add is an expensive "Pro Skater" quirk that allows you to spawn with a skateboard in your bag and have reduced self-stun times when crashing into someone + increased stun times for the person you crashed into.

I thought about giving you a trait like that if you were wearing a backwards cap (or wearing a jetpack by someone's suggestion) but I think that's a PR for the future.

@tohg7
Copy link
Contributor

tohg7 commented Jan 24, 2022

This should be ready to go for maintainer review, I haven't collected any more ideas on par with the ability to cut cake by skateboarding over it. But I might come back to it and open another PR with more stuff.
as of now :shipit:

One thing you could add is an expensive "Pro Skater" quirk that allows you to spawn with a skateboard in your bag and have reduced self-stun times when crashing into someone + increased stun times for the person you crashed into.

I thought about giving you a trait like that if you were wearing a backwards cap (or wearing a jetpack by someone's suggestion) but I think that's a PR for the future.

I think there should be a crash stun-duration bonus for crashing into someone at high speeds, like from extinguisher boosting into them.

@cacogen
Copy link
Contributor

cacogen commented Jan 26, 2022

@optimumtact optimumtact reopened this Jan 26, 2022
@itseasytosee
Copy link
Contributor Author

Thanks oranges, ready for maintainer review

Copy link
Member

@Rohesie Rohesie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hacky implementation, needs work.

code/modules/vehicles/vehicle_actions.dm Outdated Show resolved Hide resolved
code/modules/vehicles/scooter.dm Outdated Show resolved Hide resolved
code/modules/vehicles/scooter.dm Outdated Show resolved Hide resolved
code/modules/vehicles/scooter.dm Outdated Show resolved Hide resolved
code/game/atoms.dm Outdated Show resolved Hide resolved
@Rohesie Rohesie marked this pull request as draft February 1, 2022 03:17
@itseasytosee
Copy link
Contributor Author

Ok, the code is kicking my ass here. I'll clean up and get rid of the cutting feature. I promise to get back to it before I die.

@itseasytosee itseasytosee marked this pull request as ready for review February 7, 2022 22:33
code/game/atoms.dm Outdated Show resolved Hide resolved
code/game/atoms.dm Outdated Show resolved Hide resolved
Comment on lines +293 to +294
. = ..()
if(.)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this completely remove the cooldown?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was never a cooldown. The implementation of it was broken as next_ollie was never updated in relation to world.time I've removed the associated variable as well. If you are interested in adding a cooldown, although I don't see it as very necessary, it would not be difficult to implement.

var/obj/vehicle/ridden/scooter/skateboard/vehicle = vehicle_target
vehicle.obj_flags |= BLOCK_Z_OUT_DOWN
if (vehicle.grinding)
return
var/mob/living/rider = owner
var/turf/landing_turf = get_step(vehicle.loc, vehicle.dir)
rider.adjustStaminaLoss(vehicle.instability*2)
rider.adjustStaminaLoss(vehicle.instability* 0.75)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this buff entirely necessary? Can you explain why?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The original stamina cost was too high, the base skateboard can only grind over 4 tables maximum before having to take a break and even less if you want to chain skateboard grinds one after another because of the stamina cost of the ollie.
Makes it feel cooler, it should have always been this way.

@Rohesie Rohesie marked this pull request as draft February 8, 2022 12:21
itseasytosee and others added 3 commits February 8, 2022 08:54
@itseasytosee itseasytosee marked this pull request as ready for review February 8, 2022 15:13
@Rohesie Rohesie added the Balance Changes to functionality that modifies how effective certain methods are at powergaming label Feb 8, 2022
@Rohesie
Copy link
Member

Rohesie commented Feb 8, 2022

Something tells me I'll regret this one.

@Rohesie Rohesie merged commit 2468d51 into tgstation:master Feb 8, 2022
github-actions bot added a commit that referenced this pull request Feb 8, 2022
@Mothblocks Mothblocks added the Feature Exposes new bugs in interesting ways label Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet