 |
This functionality relies on the built in Slicer settings file, using the Markups/AlwaysDeleteMarkups don't show again settings key. Looking through my own .config directory, I'm not finding that string in any of the .ini files. I verified that the problem is reproducible on my current svn build. |
 |
I didn't see any other similar issues, but I didn't look hard. I just report what I experienced in my usage scenarios. |
|
|
I double checked the Markups display settings that you can save as defaults are saved to the .ini file, so it's something particular to the ctkMessageBox that's not getting saved. |
|
|
For my reference, the don't show again was added in Aug 2013: |
 |
I just looked into this issue further. The reason the selection is not persisted in the settings ini file is that the buttons used to create the deleteAllMsgBox are QMessageBox::Yes and QMessageBox::No, which have a buttonRole QDialogButtonBox::ButtonRole == YesRole. The preference is only persisted by the ctkMessageBox when the QDialogButtonBox::ButtonRole == AcceptRole. The simplest fix is to change the buttons from Yes/No to something like Ok/Cancel, which have the correct ButtonRole. On could argue though, that this issue is a failing in CTK for being so restrictive. It seems that a YesRole should be as acceptable as an AcceptRole, but there is a bit more risk involved with that change, in my opinion. I have a fix in code for either resolution. |
|
|
Integrated David's branch as svn 23315: I incorporated the same type of fix for the conversion of annotations, will integrate that as part of the Rulers xfer. |
|
|
Conversion pop up box fix: |
- Anonymous
