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

rollerblade skates item proposal #8013

Merged
merged 3 commits into from Jun 20, 2014

Conversation

Projects
None yet
3 participants
@Efimero
Copy link
Contributor

commented Jun 19, 2014

Initial commit for the proposal of rollerblades item, a type of shoe
that offers double movement speed on flat ground, but half otherwise and
has a chance to get you knocked down if used while getting attacked.
Open to suggestions.

rollerblade skates item proposal
initial commit for the proposal of rollerblades item, a type of shoe
that offers double movement speed on flat ground, but half otherwise and
has a chance to get you knocked down if used while getting attacked.
Open to suggestions.
"storage" : 0,
"symbol" : "[",
"description" : "A pair of inline skates. Very fast on flat floors, but make it hard to move on rough terrain.",
"price" : 85000,

This comment has been minimized.

Copy link
@KA101

KA101 Jun 19, 2014

Contributor

OK, I found a pair of rollerblades online for $900, but most clock in around $65-100. (Yes, that's how I price armor pieces.)

This comment has been minimized.

Copy link
@Efimero

Efimero Jun 19, 2014

Author Contributor

I based the price on other similar items. I'm not actually sure how it converts to in-game money, and also, I'm not sure how prices in-game relate to irl prices, but if you say it should be 10 times less, I trust you. =3

@@ -441,6 +441,10 @@ int player::get_dodge()

int player::dodge_roll()
{
if (is_wearing("roller_blades") && one_in(get_dex() / 3)) {
if(!has_disease("downed")) {add_msg("Fighting on wheels is hard!");}

This comment has been minimized.

Copy link
@KA101

KA101 Jun 19, 2014

Contributor

I'll check this, but if it doesn't already, you'd need to tell the player if they're getting knocked down. Probably worth involving Dodge here too (and granting Dodge practice for the roll) so characters can improve their blade-fighting skills.

(Hockey players IRL seem to maneuver pretty well.)

This comment has been minimized.

Copy link
@Efimero

Efimero Jun 19, 2014

Author Contributor

The downed disease takes care of saying you've been knocked down. It also adds a description of you bing on the floor in the status screen effects tab.
Even if you're knocked down by a bad step, your dodge is not interrupted, so you can still get away while on the floor. It takes more time for you to get up, but you can still fight as normal. This is a limitation of the downed disease effect, which is, apparently, very tame if you are either dexterous or strong. If you're not under very low stats, the downed disease will very rarely affect you by removing 100 move points.
I don't know if this design was intended differently, but it seemed to me like the easiest way and the most appropriate of implementing falling to the ground while conscious.

Efimero added some commits Jun 20, 2014

added spawning and some fixes
added spawning groups and fixed price and dodge skill dependency
@@ -2614,6 +2619,7 @@
["boots_western", 10],
["clogs", 1],
["leathersandals", 10]
["roller_blades", 20],

This comment has been minimized.

Copy link
@KA101

KA101 Jun 20, 2014

Contributor

Minor point, I fixed in the merge: comma after every item but the last in the list. ;-)

@KA101 KA101 merged commit 1657710 into CleverRaven:master Jun 20, 2014

@@ -441,6 +441,10 @@ int player::get_dodge()

int player::dodge_roll()
{
if (is_wearing("roller_blades") && one_in((get_dex() + skillLevel("dodging"))) / 3 {

This comment has been minimized.

Copy link
@vache

vache Jun 20, 2014

Contributor

The outermost ) should be moved to the other side of the 3.

This comment has been minimized.

Copy link
@KA101

KA101 Jun 20, 2014

Contributor

Already taken care of. ;-)

@Efimero Efimero deleted the Efimero:rollerblades branch Jun 20, 2014

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.