-
Notifications
You must be signed in to change notification settings - Fork 30
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
"Pa" works but "Ma" doesn't #183
Comments
I actually implemented this, but reverted it along with mama and other things in #145. The main reason for the removal was that I'm just not great at writing description text, let alone text that fits with the style of the original author, and without good descriptions, new objects are kind of pointless. Also, every new createable object tends to allow the creation several other new ones, which will also need descriptions, and so on. Every object makes the game a tiny bit slower, and if you keep adding new ones, the slowdown will eventually become noticeable. Still, if you have a really good description of a ma and @emshortif approves, I will consider it. |
Fair enough. I'd think the same would be true for "pa", and it's mostly the parallel that makes this particular example feel odd (especially since, as in your previous implementation, it doesn't take much of a tweak to the "pa" description to have a "ma" one). But that's reasonable enough regardless, it's not otherwise a particularly interesting thing. |
I tend to agree, even if I wish the description was better. And unless I'm missing something, it will still not be possible to create a mama, so we don't need to re-add her. Does @emshortif have an opinion on this? |
Testable via YAM -> MAY -> MA.
The text was updated successfully, but these errors were encountered: