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

Mapping 3D preview bugs #12

Open
peteihis opened this issue Aug 19, 2019 · 5 comments
Open

Mapping 3D preview bugs #12

peteihis opened this issue Aug 19, 2019 · 5 comments

Comments

@peteihis
Copy link
Contributor

  • The previewer has a bug, that the wireframe presentation and the rendered presentation don't match. The wire frame appears much larger. (In my test case at least). This remind me of a similar mismatch bug in one GLCanvasDrawer, that was found (and fixed) earlier. (Juts a tgeory, but the may be calculations happening that are actually working around the bug that is no longer there.)

  • This is not really a bug, but it is possible to move the preview piece off the image.

@Sailsman63
Copy link
Member

Can you still duplicate this with the current version?

@peteihis
Copy link
Contributor Author

peteihis commented Oct 5, 2020

OOPS! <-- -->

@peteihis
Copy link
Contributor Author

..So....

The 1st bullet has been fixed.

It is still possible to move the object outside the view and never find it again. (In the AoI mapping previews there is a pop-up menu, but that has not been copied here.)

Also the mouse action depth does not seem to match the depth where the object is, but then the function is different from the usual views anyway: Instead of having the camera rotate around the object here you actually move and rotate the object in front of the camera.

@Sailsman63
Copy link
Member

I was mostly concerned about the first point.

The second is less a bug, and more missing signposts. perhaps the field that is shown below the item should have a pattern?

Its been in the back of my mind for a while that all of the mini-preview windows have their own mouse-handling code, when they should probably just re-use Space mode, but that's a separate project.

@peteihis
Copy link
Contributor Author

they should probably just re-use Space mode, but that's a separate project.

I think there is a reason why the preview windows come with mouse handling of their own but it's true there are a few things that could be improved. - For one thing scroll wheel action should be added. I have had a look at those many times but never ended up doing anything about it.

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

No branches or pull requests

2 participants