Skip to content

Shore Ocean PR (Unfinished) - #41

Draft
ThreeEelsinaJellyfishCoat wants to merge 335 commits into
Toby222:devfrom
ThreeEelsinaJellyfishCoat:dev
Draft

ThreeEelsinaJellyfishCoat wants to merge 335 commits into
Toby222:devfrom
ThreeEelsinaJellyfishCoat:dev

Conversation

@ThreeEelsinaJellyfishCoat

@ThreeEelsinaJellyfishCoat ThreeEelsinaJellyfishCoat commented Jan 13, 2026 •

Copy link
Copy Markdown

To-Do list can be found on the Shore Ocean thread on the Discord server.

@ThreeEelsinaJellyfishCoat

Copy link
Copy Markdown
Author

Just saw what happened at Glowkate’s lol

Comment thread js/data/homemessages.js Outdated
shore: [
{
name: "shore-default",
message: "The surface of the water is painfully close, yet it stretches vast.<br>Looking into the distance, that seems the least of your problems.",

@glowkate glowkate Jan 22, 2026 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I don't think that the <br> is needed. From what I've seen the text wraps around on its own. I may be wrong about that though.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The, uh… the what?
I’m afraid whatever is after ‘the’ isn’t showing for me.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Ohh! The <br>! I was partially aware that it might do that, but that’s the start of a new topic, and topic change = line change.

Comment thread js/data/upgrades.js Outdated
},
},
shore: {
template: {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I believe the indenting on this is slightly wrong.

Comment thread js/data/upgrades.js
desc: "What is that sandy cloud? What does it do? We should go inside it.",
researchedMessage: "It's a sandstorm. The one where the rough grains of sand come from. To say it's torture is an understatement.",
effectDesc: "If we ever escape this... this barrage, we'll never go near it again.",
cost: {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm concerned that there are too many different types of resources needed for this upgrade. Not sure how good that'll look visually.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yes, I also think that’s a bit too much. Perhaps I’ll remove crystals.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

And science

Comment thread js/data/upgrades.js
desc: "We need to go back. This hurts.",
researchedMessage: "No. No, no, no, no, no. It's gone. It's covered.",
effectDesc: "Our previous location has been consumed by the sandstorm.",
cost: {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same concern here. I'd limit the amount of different resources it costs to 3 or less.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

No problemo!

Comment thread js/data/upgrades.js
incomeMultiplier: {
crab: 2,
},
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Incorrect indentation here

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Indentation? May I ask what that is?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Is it the commas and lines and stuff?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

It's the whitespace at the start of a line

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Oh, thank you!

Comment thread js/data/upgrades.js Outdated
underwaterChemistry: {},
thermalVents: {},
laserRays: {},
stabilization: {},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

From my memory, having upgrades that didn't previously exist be 'imported' like this causes issues.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hmmm… perhaps Murk Assessment could be Seabed Geology, and Our Only Choice could be Grotto? That still leaves an (almost) effectless upgrade in between, though…
Lore-wise, it was gonna be ‘hey, let’s immediately check out this sandstorm’ with the scientists, and then, uh… stuff happens, people die, blablabla, then you burst into the Expanse where you have time to do the normal upgrades. But I suppose I could sacrifice that for doing normal upgrades before. That still works lore-wise.

Comment thread js/data/worldtypes.js Outdated
present: ["mudskipper", "driftwood", "hardbark", "coral", "seagrass", "caracara"],
tip: "Not all is as it seems. Respect the brown fish that survive the beyond.",
},
entry: "You enter a low-strung sandy sea, just over tall enough for you to fit. It's clear water on all sides, except for behind you...",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Improper indenting on this whole section

Comment thread js/main.js Outdated
Comment thread js/puns.js Outdated
"Placeholder :D",
],
eel: [
"Placeholder :D ...do I really have to do all these?",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mayhaps you do, mayhaps you don't. Depends on if it works or not.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

My main concern is: will this code being here generate a new, working button? Or will I have to code the button to be there?

@Toby222

Toby222 commented Jan 22, 2026

Copy link
Copy Markdown
Owner

npm run fix to fix most formatting

@ThreeEelsinaJellyfishCoat

Copy link
Copy Markdown
Author

Thank you, everyone! This was really helpful!

@ThreeEelsinaJellyfishCoat

Copy link
Copy Markdown
Author

@glowkate , in response to your review, I have limited the Upgrade costs. Is it better now?

This branch was successfully deployed

1 active (outdated) deployment
Preview — 9c8ddf0f Deployed Jan 16, 2026 by vercel[bot]
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.

3 participants