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

New reptilic eyes for dragons and lizans #278

Conversation

Stadler76
Copy link
Collaborator

@Stadler76 Stadler76 commented Jul 17, 2016

Step 5 of issue #269

  • I prefer to use the body part constants rather than their hardcoded numeric values, so I replaced some of them in dragonScore() and lizardScore()
  • TONGUE_SNAKE now grants points towards your lizardScore()
  • Tiny fix in emberTFs: "[...]You have as many horns as a lizandragon can grow."
  • new PlayerHelper-methods: hasDragonHorns() and hasReptileEyes()
  • And business as usual: I tested/debugged it with hummanus, reptilum and drakes flower.

**Step 5 of issue Kitteh6660#269**
- I prefer to use the body part constants rather than their hardcoded numeric values, so I replaced some of them in dragonScore() and lizardScore()
- Tiny fix in emberTFs: "[...]You have as many horns as a <del>lizan</del><ins>dragon</ins> can grow."
- new PlayerHelper-methods: hasDragonHorns() and hasReptileEyes()
{
outputText(" Of course, no feline face would be complete without vertically slit eyes");
outputText(!player.hasReptileEyes() ? "." : ", although they come with a second set of eyelids, which is somewhat unusual for a cats face.");
}
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That part was somewhat nasty. I guess adding EYES_CAT later would make things easier in the future.
Adding an eyeCount to the player vars, adding hasSpiderEyes() to Creature.as (Similar to isTaur()) and deprecating EYES_FOUR_SPIDER_EYES would be an idea, too. So cats can still have 4 vertically slitted eyes ^^

case EYES_LIZARD: outputText(" those of a lizard with vertically slitted pupils and green-yellowish irises. They"); break;
}
outputText(" come with the typical second set of eyelids, allowing you to blink twice as much as others.");
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

And again the cats face. Well, this time it was easy to implement.

@Kitteh6660 Kitteh6660 merged commit fb4293f into Kitteh6660:master Jul 18, 2016
@Stadler76 Stadler76 deleted the Issue-#269-step-5-(New-eyes-for-reptilic-races) branch July 18, 2016 07:00
aimozg pushed a commit to aimozg/Corruption-of-Champions-Mod that referenced this pull request May 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants