Loving the latest Validation tool, but I found a few improvements. Putting it all here in one feature request for now.
1. Make Validation issues sortable by issue
Background: I recently ran the validation over an old application (DQMH 5). It seemed to sort things by VI. It would be nice to be able to sort the list by issue, so that you can easily tick off each issue one at a time. You can do that at the moment by multiple clicking separate lines, but it would be nicer if it was grouped together.
2. Put a tip somewhere on the panel: "Tip: Double click a row to show the issue."
3. Double click didn't seem to work every time. I tried it on a few issues, and nothing seemed to open or do anything.
4. Display a warning dialog after the Fix All Issues button is pressed. I accidentally pressed Fix All Issues instead of Fix Selected Issues, and it just blazed away. Suggest a dialog: "This will fix all issues listed here. Do you want to continue?"
5. Fix All Issues button location - I prefer to use Fix Selected Issues, and it's so tempting to just press Fix All Issues. Not sure what the answer is here, but perhaps Fix Selected Issues should be where Fix All Issues button is. This is probably quite subjective. If we implement previous suggestion, then this may not matter as much.
6. Where an issue will actually potentially delete/remove code from an existing VI, I suggest rather than a "Note:" description, separate the text and call it a "Warning" or something similar, and colour in red if possible so that it stands out. I can't remember which issue this was related to sorry, but it was only one issue.
7. When we get this:
The block diagram of RT Event Logger.lvlib:Main.vi contains a Register For Events function labeled "DQMH_REG_EVENTS". The source of the RT Event Logger Request Events terminal on this function is named "Event Logger Request Events". In order for the DQMH scripting tools to work correctly, change the source of this wire (usually a subVI output or a user event constant) to have a label of "RT Event Logger Request Events" instead.
This information was redundant by the time I'd already fixed the sub VI. Perhaps add an extra note to this description: "Note: This may become automatically resolved by addressing the associated Sub VI" or something to that affect.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.