Ticket #7162 (closed: fixed)
Turn off LOD for SplatterPlot view
Reported by: | Michael Reuter | Owned by: | Michael Reuter |
---|---|---|---|
Priority: | major | Milestone: | Release 2.6 |
Component: | GUI | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Owen Arnold |
Description
The level-of-detail option should be off by default for the SplatterPlot view just like it is for the ThreeSlice view.
Change History
comment:2 Changed 7 years ago by Michael Reuter
Refs #7162. Turn off LOD by default for SplatterPlot view.
Changeset: 517845af2235fa4c57bba5f0b517b45353c7adf6
comment:3 Changed 7 years ago by Michael Reuter
Merge branch 'feature/7162_lod_off_splatterplot' into develop
comment:4 Changed 7 years ago by Michael Reuter
Merge branch 'feature/7162_lod_off_splatterplot' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:5 Changed 7 years ago by Michael Reuter
Merge branch 'feature/7162_lod_off_splatterplot' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:6 Changed 7 years ago by Michael Reuter
- Status changed from accepted to verify
- Resolution set to fixed
Branch is here.
To test:
- Open any MDEventWorkspace in the VSI
- Switch to SplatterPlot view
Go to the View menu and see that the Level-of-Detail entry is unchecked.
comment:7 Changed 7 years ago by Owen Arnold
- Status changed from verify to verifying
- Tester set to Owen Arnold
comment:9 Changed 7 years ago by Owen Arnold
I merged the feature branch into master anyway. It works as advertised. If the checkbuild can be reapplied, I'll be able to pass this ticket. One minor problem with the way this works is that it can 'forget' the non-default settings from the other views. For example:
- On the standard view turn the LOD to Off (non default)
- Switch to splatter plot view. The LOD is also switched off.
- Now go back to the standard view, and the VSI turns the LOD back on for the StandardView
Not sure whether this behaviour is desirable or non-desirable, but it could be ammended under a separate ticket if required.
comment:10 Changed 7 years ago by Michael Reuter
Hmm. Have to think about that one.
comment:13 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8008