Skip to content

Civero Refactor - #17

Open
Brackets-Coder wants to merge 1 commit into
mainfrom
civero-refactor
Open

Civero Refactor#17
Brackets-Coder wants to merge 1 commit into
mainfrom
civero-refactor

Conversation

@Brackets-Coder

Copy link
Copy Markdown
Owner

This is a quick refactor and formatting.
I was seriously going to add positional audio and realized it was already there.

What you've cooked up is pretty incredible, it needs polishing & performance though. I'll get to that later.

@Civ3ro let me know if you disagree with any of these changes.

Comment thread civero/three2.js
Comment thread civero/three2.js
@@ -2829,5 +2715,5 @@ SOFTWARE.
.catch((err) => {
console.error("Extension failed to load: ", err.message);
});

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

How do you make it look nice?
I doubt you did all of those changes manually, such small details! But that makes the polished look.
Is it a VS extension?

@Brackets-Coder Brackets-Coder Apr 5, 2026

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Nope, VS Code has JS formatting built-in as far as I'm aware. (Ctrl + A > Right click > Format Document). I also used Prettier npx prettier --write [file]. Then the rest was done manually

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.

2 participants