Conversation
| { | ||
| exportPath = StandaloneFileBrowser.StandaloneFileBrowser.SaveFilePanel("Save patch as...", Application.dataPath, "TerrainPatch", "optoctreepatch"); | ||
| if (string.IsNullOrEmpty(exportPath)) return; | ||
| if (!exportPath.EndsWith(".optoctreepatch")) exportPath += ".optoctreepatch"; |
There was a problem hiding this comment.
This wont fix the issue with #19. issue #19 is that the file browser window on Linux for some reason cannot automatically fill in this extension. Its also an issue with imports being unable to sort by file type :/
The only way to fix this is within the unmanaged C++ library of UnityStandaloneFileBrowser, which i have zero knowledge to fix
Tho this is good error checking and is not a terrible bandaid solution given the situation, just prob not the ideal way to fix it
| string combinedString = (materialNameRequest.asset as TextAsset).text; | ||
|
|
||
| string[] lines = combinedString.Split(new[] {Environment.NewLine}, StringSplitOptions.None); | ||
| string[] lines = combinedString.Replace("\r\n", "\n").Split('\n'); |
There was a problem hiding this comment.
If this actually works it will save me SO much time. It feels like it shouldn't work tho given that Enviroment.NewLine should account for platform differences at runtime.
If you can test this and it does work then I will be amazed
|
|
||
| //Apply | ||
| transform.rotation = horiz * rotation * vert; | ||
| transform.rotation = Quaternion.Euler(pitch, yaw, 0f); |
There was a problem hiding this comment.
if this is tested and the camera feels roughly (or exactly) the same, then lgtm
2 participants
Three small self-contained bug fixes. Closes #48, #24, #19.