Skip to content

Conversation

@dicedpixels
Copy link
Contributor

@dicedpixels dicedpixels commented Sep 17, 2025

Still a hater, but surprisingly it works well enough.

This PR:

  • Adds a page for setting up Visual Studio Code.
  • Couple of tips for searching classes and generating bytecode.
  • Split the "Setting up a Development Environment" page and move the Intellij part to its own page.
  • Re-arrange sidebar to fit both IDEs.

Should close #176. Given how few use Eclipse, it's not worth the effort.

@dicedpixels dicedpixels requested a review from a team as a code owner September 17, 2025 15:35
@netlify
Copy link

netlify bot commented Sep 17, 2025

Deploy Preview for nimble-elf-d9d491 ready!

Name Link
🔨 Latest commit a6e3c4a
🔍 Latest deploy log https://app.netlify.com/projects/nimble-elf-d9d491/deploys/68d2d20ce67216000839ff1e
😎 Deploy Preview https://deploy-preview-360--nimble-elf-d9d491.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

cassiancc
cassiancc previously approved these changes Sep 17, 2025
@its-miroma its-miroma added new-content A new page or set of pages todo:verify This needs to be reviewed priority:low This is nice to have, but it can wait a little labels Sep 17, 2025
@its-miroma

This comment was marked as outdated.

@its-miroma its-miroma changed the title Setting up a Development Environment Using Visual Studio Code "Setting up a Development Environment": VSCode Sep 17, 2025
@dicedpixels

This comment was marked as resolved.

@its-miroma

This comment was marked as outdated.

@dicedpixels

This comment was marked as resolved.

@cassiancc

This comment was marked as resolved.

Copy link
Member

@its-miroma its-miroma left a comment

Choose a reason for hiding this comment

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

The VS Code-specific information is all neatly centralized in this article, while the same for IDEA is scattered everywhere in the Docs...

see:

  • getting-started/setting-up-a-development-environment/*.md
  • getting-started/creating-a-project.md#importing-the-project
  • getting-started/launching-the-game.md
  • ide-tips-and-tricks.md

I don't think it has to be done in this PR, but I'm for creating getting-started/idea and getting-started/vscode with subpages for each topic.

I'm not saying all the docs have to be split, just the IDE-specific pages listed above.

@its-miroma its-miroma changed the title "Setting up a Development Environment": VSCode "Setting up a Development Environment": VS Code Sep 23, 2025
@its-miroma its-miroma added todo:cleanup This should be cleaned up and removed todo:verify This needs to be reviewed labels Sep 23, 2025
@dicedpixels

This comment was marked as outdated.

@its-miroma
Copy link
Member

I'd put "Setting Up a Development Environment" last in the sidebar:

getting-started/:
  - index.md  # Introduction to Fabric and Modding
  - creating-a-project.md
  - project-structure.md
  - setup-ide/:
    - idea/:
      - opening-the-project.md
      - launching-the-game.md
      - ...
      - tips-and-tricks.md
    - vscode/:
      - ...

Copy link
Member

@its-miroma its-miroma left a comment

Choose a reason for hiding this comment

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

Here's another possible solution for the headings, what do you think?

its-miroma
its-miroma previously approved these changes Sep 23, 2025
@cassiancc cassiancc merged commit 8d7d92f into FabricMC:main Sep 23, 2025
8 checks passed
@its-miroma its-miroma added stage:ready and removed todo:cleanup This should be cleaned up labels Sep 23, 2025
its-miroma added a commit to dicedpixels/fabric-docs that referenced this pull request Sep 24, 2025
* upstream/main:
  Update rendering concepts page (FabricMC#362)
  "Setting up a Development Environment": VS Code (FabricMC#360)
  Running Game Tests In Github Actions (FabricMC#338)
  Items: Item Models (FabricMC#324)
  Call `.useBlockPrefixedTranslationKey()` (FabricMC#358)
  Clarify that `mod-id` placeholders must be customized (FabricMC#348)
  replace remaining mentions of tickDelta with tickProgress (FabricMC#361)
  Project location (FabricMC#326)
  improve the first items docs to state what's needed for item appearance, link fix (FabricMC#359)
kevinthegreat1 pushed a commit to kevinthegreat1/fabric-docs that referenced this pull request Oct 14, 2025
* add page for setting up vscode, tips and re-arrange sidebar

* address review comments

* address review comments

* address review comments

* address review comments

(cherry picked from commit 8d7d92f)
kevinthegreat1 pushed a commit to kevinthegreat1/fabric-docs that referenced this pull request Oct 14, 2025
* add page for setting up vscode, tips and re-arrange sidebar

* address review comments

* address review comments

* address review comments

* address review comments

(cherry picked from commit 8d7d92f)
kevinthegreat1 pushed a commit to kevinthegreat1/fabric-docs that referenced this pull request Oct 15, 2025
* add page for setting up vscode, tips and re-arrange sidebar

* address review comments

* address review comments

* address review comments

* address review comments

(cherry picked from commit 8d7d92f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-content A new page or set of pages priority:low This is nice to have, but it can wait a little

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Setting up a Development Environment": Support VS Code

3 participants