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

Change default value of "Show description (if present)" preference #134

Closed
mathewbutler opened this issue Jan 7, 2021 · 2 comments · Fixed by #149
Closed

Change default value of "Show description (if present)" preference #134

mathewbutler opened this issue Jan 7, 2021 · 2 comments · Fixed by #149
Assignees
Milestone

Comments

@mathewbutler
Copy link

mathewbutler commented Jan 7, 2021

(thanks again for the time and effort put into this extension - it's a real productivity improvement for database testing using Oracle SQL Developer)

Following on from this discussion in the utPLSQL slack channel;

The 1.3.1 release makes the tree-table view of tests the default (the use of this view being controlled via the “Show suites (hierarchical view of tests)" preference). In the tree table view, an improvement would be to see the descriptions associated with tests and context's if these are present. This can be controlled by the preference "Show description (if present)", but the default of this preference is "disabled".

This enhancement request, is to change the default value of the preference "Show description (if present)" to be "enabled".

Also, in the context of the introduction of the tree-table view, it's worth considering whether the other preference defaults are still sensible - though these aren't the subject of this enhancement request.

@PhilippSalvisberg PhilippSalvisberg self-assigned this Jan 7, 2021
@PhilippSalvisberg PhilippSalvisberg added this to the v1.4.0 milestone Jan 7, 2021
@PhilippSalvisberg
Copy link
Member

Thanks Mathew.

Maybe all of the Realtime Reporter options in the preferences should be enabled like this:

image

Impact for initial installations:

  • The three counters (disabled, warning, info) will probably add just an additional line.
  • Indicators (warning info) will leave less space for description.

Disabling information is probably more intuitive than enabling information.

@mathewbutler
Copy link
Author

On the basis (based on slack discussion) that the state of any configured options changed by the user in a previous install are carried forward, then I think these defaults you have listed make sense.

PhilippSalvisberg added a commit that referenced this issue Feb 26, 2022
…according #134

checked instead of unchecked for
- Show disabled counter?
- Show warning counter?
- Show info counter?
- Show warning indicator?
- Show info indicator?
- Show description (if present)?
PhilippSalvisberg added a commit that referenced this issue Feb 26, 2022
…ording

checked instead of unchecked for
- Show disabled counter?
- Show warning counter?
- Show info counter?
- Show warning indicator?
- Show info indicator?
- Show description (if present)?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants