Add Check Invalid Rotation Plugin#785
Open
PlanesZwalker wants to merge 9 commits intoJannisX11:masterfrom
Open
Add Check Invalid Rotation Plugin#785PlanesZwalker wants to merge 9 commits intoJannisX11:masterfrom
PlanesZwalker wants to merge 9 commits intoJannisX11:masterfrom
Conversation
Contributor
PlanesZwalker
commented
Oct 18, 2025
PlanesZwalker
commented
- Detects non-standard rotations in models before export
- Shows warning dialog with element names and suggested fixes
- Auto-fix functionality for rotations to nearest standard values
- User choice to continue export or cancel to fix first
- Supports multiple rotation fix options
- Compatible with Blockbench 4.12.6+ and Minecraft Bedrock standards
- Automatically detects non-standard rotations in models - Shows warning dialog before export to prevent Minecraft display issues - Auto-fix functionality to correct rotations to nearest standard values - User choice to continue export or cancel to fix rotations first - Compatible with Blockbench 4.12.6+
PlanesZwalker
changed the title
JannisX11
requested changes
Oct 26, 2025
Owner
JannisX11
left a comment
JannisX11
left a comment
There was a problem hiding this comment.
Left comments on a few issues that I've found.
Apart from that, the issue that I noticed was that no check was performed when saving via Ctrl + S, or File > Save Model.
Btw, in case you didn't see, Minecraft recently lifted the 22.5 degree limit for models, and on bedrock it has never existed in the first place. So I assume this is just intended for legacy versions, any maybe for stylistic consistency if the style requirement is to align with these rotations?
Also, for anything validation-related I can also recomment the Validator API, it can display issues in real time in the UI so could be an addition: https://web.blockbench.net/docs/classes/custom_validator.ValidatorCheck.html
| icon: 'warning', | ||
| version: '1.0.0', | ||
| variant: 'both', | ||
| tags: ['minecraft', 'rotation', 'validation', 'export'], |
Owner
There was a problem hiding this comment.
You can specify up to 3 tags. Make sure they are formatted for display in UI, in this case they should start with an upper case letter.
| return nonStandardRotations; | ||
| } | ||
|
|
||
| function applyRotationFix(rotation) { |
Comment on lines
414
to
415
| if (Interface && Interface.updateViewport) { | ||
| Interface.updateViewport(); |
| Interface.updateViewport(); | ||
| } | ||
|
|
||
| if (Outliner && Outliner.update) { |
| Outliner.update(); | ||
| } | ||
|
|
||
| if (Project && Project.update) { |
| } | ||
|
|
||
| setTimeout(() => { | ||
| if (Interface && Interface.updateViewport) { |
- Remove non-compatible API calls from refreshScene() function - Simplify about.md by removing duplicate version info and repetitive benefits - Streamline changelog.json to avoid feature duplication - Address feedback from @JannisX11 for better maintainability
- Target specific version range (1.8 - 1.18.x) instead of broad 1.8+ - Add disclaimer about Minecraft 1.19+ potentially relaxing restrictions - Update changelog format to match schema (object instead of array) - Reduce tags to 3 maximum (removed 'minecraft' tag) - Improve version specificity and honesty about plugin scope - Update both English and French translations consistently
… fix uninstall cleanup - Removed duplicate save integration methods causing multiple dialogs - Cleaned up code for production (removed comments, optimized formatting) - Fixed uninstall cleanup to properly restore BarItems functions - Individual 'Corriger' buttons now work correctly - Reduced file size by 24% (1257 → 860 lines) - All functionality tested and working
Contributor
Author
|
Here is my 'cleaned' and improved version ;) |
…alker/blockbench-plugins-Translation-planes-gizmo into rotation-checker-plugin # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.