Skip to content

Conversation

@kevinthegreat1
Copy link
Contributor

I've updated the code. It's still a bit confusing, but it should be at least correct and guide people on where to look.

Working impl of rendering concepts in 1.21.8

@kevinthegreat1 kevinthegreat1 requested a review from a team as a code owner September 18, 2025 18:20
@netlify
Copy link

netlify bot commented Sep 18, 2025

Deploy Preview for nimble-elf-d9d491 ready!

Name Link
🔨 Latest commit 2d1c3f0
🔍 Latest deploy log https://app.netlify.com/projects/nimble-elf-d9d491/deploys/68d2f4105c17ce00086555f7
😎 Deploy Preview https://deploy-preview-362--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 18, 2025
@its-miroma its-miroma added correction Something isn't right on a page todo:verify This needs to be reviewed new-version Related to a newer version of Minecraft priority:medium This should be addressed relatively quickly labels Sep 18, 2025
@kevinthegreat1
Copy link
Contributor Author

I am going to merge this if there are no other reviews.

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.

slight reword, LGTM!

kinda confusing, but that's passable

Co-authored-by: Miroma <136986257+its-miroma@users.noreply.github.com>
@its-miroma its-miroma added stage:ready and removed todo:verify This needs to be reviewed labels Sep 23, 2025
@kevinthegreat1 kevinthegreat1 merged commit 054180d into FabricMC:main Sep 23, 2025
8 checks passed
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 kevinthegreat1 deleted the rendering-concepts branch October 14, 2025 17:02
kevinthegreat1 added a commit to kevinthegreat1/fabric-docs that referenced this pull request Oct 14, 2025
* Update rendering concepts page

* Uncomment note in hud.md

* Apply suggestions from code review

---------

Co-authored-by: Miroma <136986257+its-miroma@users.noreply.github.com>
(cherry picked from commit 054180d)
kevinthegreat1 added a commit to kevinthegreat1/fabric-docs that referenced this pull request Oct 14, 2025
* Update rendering concepts page

* Uncomment note in hud.md

* Apply suggestions from code review

---------

Co-authored-by: Miroma <136986257+its-miroma@users.noreply.github.com>
(cherry picked from commit 054180d)
kevinthegreat1 added a commit to kevinthegreat1/fabric-docs that referenced this pull request Oct 15, 2025
* Update rendering concepts page

* Uncomment note in hud.md

* Apply suggestions from code review

---------

Co-authored-by: Miroma <136986257+its-miroma@users.noreply.github.com>
(cherry picked from commit 054180d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

correction Something isn't right on a page new-version Related to a newer version of Minecraft priority:medium This should be addressed relatively quickly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants