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

Fixes grammar and description. #4789

Merged
merged 3 commits into from Dec 3, 2013

Conversation

Projects
None yet
4 participants
@Rivet-the-Zombie
Copy link
Member

commented Dec 1, 2013

No description provided.

@swwu

This comment has been minimized.

Copy link
Contributor

commented Dec 1, 2013

No real substantive comments from me but this PR is pure gold.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Dec 1, 2013

XD

Wasn't sure about that one... Nice.

@@ -1936,7 +1936,7 @@
"symbol": "(",
"color": "dark_gray",
"name": "RM802 grenade launcher",
"description": "The Rivtech RM802 pump-action 40mm grenade launcher is designed to be a step ahead of single-shot grenade launchers by allowing sustained heavy firepower. However, its capacity is limited must be used with careful planning.",
"description": "The Rivtech RM802 pump-action 40mm grenade launcher was designed to be a step ahead of single-shot grenade launchers by allowing sustained heavy firepower. However, its capacity is still fairly limited so it must be used with careful planning.",

This comment has been minimized.

Copy link
@freezerbunny

freezerbunny Dec 2, 2013

Contributor

However, its capacity is still fairly limited
to
However, its capacity is fairly limited???

Gotta minimize dem words!

This comment has been minimized.

Copy link
@KA101

KA101 Dec 2, 2013

Contributor

In game-messages, maybe--Wuzzy makes a valid point about size diffs in translation. But in item descriptions, where you have pretty much a whole window?

This comment has been minimized.

Copy link
@freezerbunny

freezerbunny Dec 2, 2013

Contributor

If a word contributes minimal extra information or nothing at all to a sentence, then it doesn't need to be used.

The above with that principle once more:
If a word contributes little to a sentence then it doesn't need to be used.

It's all about effective style which, in the long run, reads a lot better and will lend the game that polished feel. It is nitpicking, but I think it's so easy to change that it should be pointed out.

freezerbunny added a commit that referenced this pull request Dec 3, 2013

@freezerbunny freezerbunny merged commit 88f162d into CleverRaven:master Dec 3, 2013

1 check passed

default This has been rescheduled for testing as the 'master' branch has been updated.

@Rivet-the-Zombie Rivet-the-Zombie deleted the Rivet-the-Zombie:SPOON-FIX branch Dec 4, 2013

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.