-
-
Notifications
You must be signed in to change notification settings - Fork 8
Rotate repair states based on the rotation of the sign #51
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.
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
Conversation
|
How will this behave if a craft has multiple repair signs that face different directions? |
Multiple repair signs with different repair states don't affect each other at all since they use different schematics. I actually had two signs like this when I was testing it and I could use them interchangeably just fine. Multiple repair signs using the same repair state never worked as far as I know (and still don't) since there's no way to know which sign saved the schematic. |
Can you elaborate on this point? The schematic should not be changed depending on what rotation or state of the sign is used. |
The schematic is not changed permanently, but when the "clipboard" is rotated to align with the craft its dimensions and coordinates change. Any precalculated values would have to be rotated along with it which in my mind defeats the point of precalculating them in the first place. |
Fixes an edge case that occurred if a repair sign was precisely on the craft's center of rotation. The issue occurred if a player right-clicked the sign once, rotated the craft, and then right-clicked the sign again in quick succession. This caused the repair to begin as if the craft had not been rotated.
Pretty much what was mentioned. A craft that has the same repair sign (as in same "snapshot" name) in different positions of the ship. |
This behavior has not and never will be supported by Repair. It uses the repair sign location as the origin of the repair state. |
This pull request allows crafts to be repaired regardless of their current orientation. Every possible placement of both wall signs and "floor" signs should work.
Note that it is no longer possible to precalculate schematic coordinates since the schematic will be rotated differently depending on the sign used.