Skip to content
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

updated "Lay flat 3D Viewport" option #192

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

deHarro
Copy link
Contributor

@deHarro deHarro commented Sep 8, 2024

Viewport now snaps to horizontal/vertical aligned to monitor orientation if "Lay flat..:" is selected. Viewport beforehand has to be oriented as wanted by RightMouse dragging, then saved by "Save Viewport Orientation".

Viewport now snaps to horizontal/vertical aligned to monitor orientation if "Lay flat..:" is selected.
Viewport beforehand has to be oriented as wanted by RightMouse dragging, then saved by "Save Viewport Orientation".
@deHarro
Copy link
Contributor Author

deHarro commented Sep 8, 2024

Don't know whether the changes to files "Settings.Designer.cs" and Settings.settings", respectively, have to be taken into account...

@deHarro
Copy link
Contributor Author

deHarro commented Sep 9, 2024

Additional info
One has to adjust the viewport orientation with the mouse relatively exact to be orthogonal to the monitor layout to get my code to snap.

  • Perhaps I have to adapt the math behind to be more forgiving to deviations
  • And I should change to leave the camera distance alone to keep the current image section (fixed - not pushed yet)
  • Perhaps even better: Take into account the dimensions of the currently loaded board and center the board in the viewport, not to the origin (as it is now)

- means for setting the viewport orientation corresponding to the actual situation at the router
- temporarily added ShowCameraInfo for debugging - disabled now (ShowCameraInfo="False")
first attempt to align viewport to physical router orientation was the wrong approach (used UpDirection). Correct approach is using LookDirection.
In the process of corrrection, fiddling on UpDirection is undone.
@deHarro
Copy link
Contributor Author

deHarro commented Sep 11, 2024

I overhauled the last approach and changed some points back to original (e.g. camera.UpDirection changes in last commit as of Sept. 8th).
So please discard this Sept. 8th pull request and use the pull request of Sept. 11th.

Additionally hiding window in the taskbar
checkmark "Show Status Messages" was displayed mirrored horizontally.
(sorry for the many lines for fixing that minor flaw ;)
Merge pull request martin2250#190 from deHarro/master
In Debug Options 2 additional checkboxes for ViewCube (see martin2250#178 (comment)) and CameraInfo, respectively.
@martin2250
Copy link
Owner

Hi Harald, thank you! I'll take a look soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants