-
Notifications
You must be signed in to change notification settings - Fork 39
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
bring back the Draw grid V-Space and H-Space #1443
Comments
We wont have anything against re-introducing the h and v-space controls if you are using them. They where added because we thought there might be a use for h/v space but in the end no one actually used them. Many users also found the controls strange, for some reason, and where mostly just wondering why we had those. The decision was thus to remove them and possibly replace them with something more "user friendly" when actually requested. It might be necessary to have a second look at these controls when we have some more feedback from the scientists but adding them back as they where would be fine. |
@mockoocy will work on this. |
Restores controls used for setting horizontal and vertical cell spacing for grid mesh
Restores controls used for setting horizontal and vertical cell spacing for grid mesh
Restores controls used for setting horizontal and vertical cell spacing for grid mesh
Restores controls used for setting horizontal and vertical cell spacing for grid mesh
The
V-Space
andH-Space
boxes in the Draw grid widget have been removed in this PR: #1397We are actually using these controls. We did tests doing mesh grid scans with H-Space separation last week, and it worked for us.
Can we bring them back?
As I understand, these controls are seldom used, so let's make it configurable if they are shown. I suggest to add these two UI configs:
With default values set to false for both of them.
As a side note, at MicroMAX currently we only want to show H-Space, but in the future we want to show both. Thus it would be nice to make each box individually configurable.
The text was updated successfully, but these errors were encountered: