Skip to content

Add new entity culling options in Spanish localization #314

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.

Sign up for GitHub

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

Open
Texaliuz wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add new entity culling options in Spanish localization #314

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/main/resources/assets/entityculling/lang/es_ar.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,9 @@
"text.entityculling.skipBlockEntityCulling": "Desactivar ocultamiento de bloques-entidad",
"text.entityculling.skipBlockEntityCulling.tooltip": "Desactiva por completo el ocultamiento de bloques-entidad. Útil para depurar o desactivar temporalmente el sistema.",
"text.entityculling.blockEntityFrustumCulling": "Culling de entidades de bloque por frustum",
"text.entityculling.blockEntityFrustumCulling.tooltip": "Cuando está activado, la extracción del estado de las entidades de bloque será filtrada según el frustum de la cámara. Esto puede mejorar el rendimiento en caso de que tu cargador de mods u otro mod no lo haga ya."
"text.entityculling.blockEntityFrustumCulling.tooltip": "Cuando está activado, la extracción del estado de las entidades de bloque será filtrada según el frustum de la cámara. Esto puede mejorar el rendimiento en caso de que tu cargador de mods u otro mod no lo haga ya.",
"text.entityculling.forceDisplayCulling": "Forzar culling de visualización",
"text.entityculling.forceDisplayCulling.tooltip": "Al activarlo, las entidades de visualización sin caja de colisión recibirán una caja de colisión predeterminada para efectos de culling. Esto puede mejorar el rendimiento en mapas o servidores que no configuran bien sus entidades de visualización, pero puede causar problemas visuales.",

@coderabbitai coderabbitai Bot Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Use culling-box terminology instead of collision-box terminology.

Line 26 describes a culling bounding box, but “caja de colisión” implies physical collision behavior. Use wording such as “caja delimitadora para culling” to match the implementation.

Proposed wording 
-    "text.entityculling.forceDisplayCulling.tooltip": "Al activarlo, las entidades de visualización sin caja de colisión recibirán una caja de colisión predeterminada para efectos de culling. Esto puede mejorar el rendimiento en mapas o servidores que no configuran bien sus entidades de visualización, pero puede causar problemas visuales.",
+    "text.entityculling.forceDisplayCulling.tooltip": "Al activarlo, las entidades de visualización sin caja delimitadora para culling recibirán una caja delimitadora predeterminada. Esto puede mejorar el rendimiento en mapas o servidores que no configuran bien sus entidades de visualización, pero puede causar problemas visuales.",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"text.entityculling.forceDisplayCulling.tooltip": "Al activarlo, las entidades de visualización sin caja de colisión recibirán una caja de colisión predeterminada para efectos de culling. Esto puede mejorar el rendimiento en mapas o servidores que no configuran bien sus entidades de visualización, pero puede causar problemas visuales.",
"text.entityculling.forceDisplayCulling.tooltip": "Al activarlo, las entidades de visualización sin caja delimitadora para culling recibirán una caja delimitadora predeterminada. Esto puede mejorar el rendimiento en mapas o servidores que no configuran bien sus entidades de visualización, pero puede causar problemas visuales.",
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/main/resources/assets/entityculling/lang/es_ar.json` at line 26, Update
the Spanish tooltip value for text.entityculling.forceDisplayCulling.tooltip to
replace “caja de colisión” terminology with “caja delimitadora para culling,”
while preserving the existing meaning and surrounding wording.

@tr7zw tr7zw Jul 21, 2026

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.

image This seems correct, at least trying to translate it with deepl. Can you fix that translation?

@tr7zw tr7zw Jul 21, 2026

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.

@Texaliuz

@tr7zw tr7zw Aug 14, 2026

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.

Can you try to fix that switchup? I don't want to change it myself, as I don't speak Spanish.

"text.entityculling.solidLeaves": "Hojas sólidas",
"text.entityculling.solidLeaves.tooltip": "Al activarlo, las hojas se tratarán como sólidas para efectos de culling. Esto puede mejorar el rendimiento en bosques, pero también causar problemas visuales. Útil si tenés 'See-Through Leaves' desactivado, o si usás un resource pack de hojas tupidas."
}
Loading