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

Adjust grammar in location event strings #4450

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Atermonera
Copy link
Contributor

Description of changes

I proofread most/all of these while reviewing culture stuff on Polaris, figured I'd pass them up here. Should've scrubbed off the polaris-specific lorestrings but lmk if anything seems amiss.

Why and what will this PR improve

Words hard.

Authorship

@Atermonera

Changelog

🆑
bugfix: Fixed some grammatical issues in location event strings.
/:cl:

@@ -4,27 +4,27 @@
/decl/location_event/big_game_hunters/announce(var/decl/background_detail/location)
. = "Game hunters on [location.name] "
if(prob(33))
. += "were surprised when an unusual species experts have since identified as \
. += "were surprised when an unusual species that experts have since identified as \
Copy link
Member

Choose a reason for hiding this comment

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

this was technically grammatically correct already. for more information: it's a defining relative clause, which doesn't need a relative pronoun and is not marked by commas before and after. it's not incorrect to include 'which' or 'that', but it's not strictly necessary either.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe not strictly necessary, but w/out that the sentence comes a little close to being a garden path.

Game hunters on PLACE were surprised when an (unusual species experts have since identified as THING) turned up.

I think w/out the tailing turned up this would most likely read just as well to me.
My usual approach is to read smth like this out load and if it sounds funny, it is funny.

@@ -34,12 +34,12 @@

/decl/location_event/alien_raiders/announce(var/decl/background_detail/location)
if(prob(20))
. = "The Tiger Co-operative have raided [location.name] today, no doubt on orders from their enigmatic masters. Stealing wildlife, farm animals, medical research materials and kidnapping civilians. [global.using_map.company_name] authorities are standing by to counter attempts at bio-terrorism."
. = "The Tiger Co-operative have raided [location.name] today, no doubt on orders from their enigmatic masters, stealing wildlife, farm animals, medical research materials and kidnapping civilians. [global.using_map.company_name] authorities are standing by to counter attempts at bio-terrorism."
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
. = "The Tiger Co-operative have raided [location.name] today, no doubt on orders from their enigmatic masters, stealing wildlife, farm animals, medical research materials and kidnapping civilians. [global.using_map.company_name] authorities are standing by to counter attempts at bio-terrorism."
. = "The Tiger Cooperative have raided [location.name] today, no doubt on orders from their enigmatic masters, stealing wildlife, farm animals, medical research materials and kidnapping civilians. [global.using_map.company_name] authorities are standing by to counter attempts at bio-terrorism."

co-operative is BrEng, cooperative is AmEng, not sure if we have a preference here


/decl/location_event/ai_liberation/announce(var/decl/background_detail/location)
. = "A [pick("\'REDACTED\' was detected on","S.E.L.F operative infiltrated","malignant computer virus was detected on","rogue [pick("slicer","hacker")] was apprehended on")] [location.name] today, and managed to infect [pick("\'REDACTED\'","a sentient sub-system","a class one AI","a sentient defence installation")] before it could be stopped. Many lives were lost as it systematically begin murdering civilians, and considerable work must be done to repair the affected areas."
. = "A [pick("\'REDACTED\' was detected on","S.E.L.F operative infiltrated","malignant computer virus was detected on","rogue [pick("slicer","hacker")] was apprehended on")] [location.name] today, after it managed to infect [pick("\'REDACTED\'","a sentient sub-system","a class one AI","a sentient defence installation")] before it could be stopped. Many lives were lost as it systematically begun murdering civilians, and considerable work must be done to repair the affected areas."
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
. = "A [pick("\'REDACTED\' was detected on","S.E.L.F operative infiltrated","malignant computer virus was detected on","rogue [pick("slicer","hacker")] was apprehended on")] [location.name] today, after it managed to infect [pick("\'REDACTED\'","a sentient sub-system","a class one AI","a sentient defence installation")] before it could be stopped. Many lives were lost as it systematically begun murdering civilians, and considerable work must be done to repair the affected areas."
. = "A [pick("\'REDACTED\' was detected on","S.E.L.F operative infiltrated","malignant computer virus was detected on","rogue [pick("slicer","hacker")] was apprehended on")] [location.name] today, and managed to infect [pick("\'REDACTED\'","a sentient sub-system","a class one AI","a sentient defence installation")] before it could be stopped. Many lives were lost as it systematically began murdering civilians, and considerable work must be done to repair the affected areas."

disagree with and -> after it, especially since "rogue hacker" is an option.
begun is incorrect here, it should be began

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fair point on plural/singular pronoun, hadn't spotted that. I still kinda want to keep the timing denoted by after b/c although it's clear from reading the sentence, and feels a little off to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, but the before hiding after the second pick() carries that time info, so and is fine.

@out-of-phaze out-of-phaze added the ready for review This PR is ready for review and merge. label Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for review and merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants