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

3D Viewer light source coordinates text field needs improvement #2250

Open
acardona opened this issue Mar 24, 2023 · 1 comment
Open

3D Viewer light source coordinates text field needs improvement #2250

acardona opened this issue Mar 24, 2023 · 1 comment

Comments

@acardona
Copy link
Contributor

First, the text box expects coordinates like 30000, 40000, 10000 which isn't obvious. Best to split into three text fields.
Second, the Y axis starts at the top rather than at bottom left, which isn't obvious. It would be best to make it start at bottom left, or to at least have the tooltip text indicate so.
Third, can't save them for future uses of the 3D Viewer. Would be very nice to do so. Either by staying on, or by exporting the whole 3D Viewer state (volumes, floors, background) minus the skeletons.

@tomka
Copy link
Contributor

tomka commented Apr 27, 2023

Good points, the current documentation and UI is certainly confusing. I just have updated the placeholder to "X, Y, Z" and improved the tooltip text. Besides besides also explaining the expected format, it also talks about the coordinate system and it being right-handed to map better to the 2D viewer (where the origin is also in the upper left corner) with Z going "inwards". Since there are other input fields that also take full coordinates, I'll leave the single input field for now and hope to have reduced the confusion at least somewhat.

The setting can already be stored along with all other settings by saving the widget state: click on the window icon in the blue 3D Viewer toolbar and click on "Save settings". The current settings of that widget will now be the default for all new 3D Viewers:
image

Many widgets support this option for settings storage.

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

No branches or pull requests

2 participants