key icon indicating copy to clipboard operation
key copied to clipboard

HacKeYthon: Improved Taclet Options

Open tobias-rnh opened this issue 1 year ago • 13 comments

Related Issue

This pull request addresses #3414.

Intended Change

Remove the old dialog to change taclet options. Taclet options can now be accessed through the new unified settings dialog ("Options" -> "Show Settings" -> "Taclet Options"). Additionally, a bug was fixed where a warning header did not disappear, even after a proof was loaded.

As of now, the radio buttons selected are still only the default ones and not the options selected in the current proof.

Type of pull request

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] Refactoring (behaviour should not change or only minimally change)
  • [ ] New feature (non-breaking change which adds functionality)
  • [x] Breaking change (fix or feature that would cause existing functionality to change) Old Taclet Options are not accessible through the options tab directly anymore
  • [x] There are changes to the (Java) code
  • [ ] There are changes to the taclet rule base
  • [ ] There are changes to the deployment/CI infrastructure (gradle, github, ...)
  • [ ] Other:

Ensuring quality

  • [x] I made sure that introduced/changed code is well documented (javadoc and inline comments). Really only inline comments
  • [ ] I made sure that new/changed end-user features are well documented (https://github.com/KeYProject/key-docs).
  • [ ] I added new test case(s) for new functionality.
  • [x] I have tested the feature as follows: I manually inspected the menus and how they behaved before and after loading a proof
  • [ ] I have checked that runtime performance has not deteriorated.

Additional information and contact(s)

As a reference, how things were looking before and now:

The options tab previously had an entry to get to taclet options directly. old_options

Now, this is not the case anymore since the options are now contained in a unified settings dialog. new_options

Before, even when a proof was loaded, there was still a header shown in the taclet options stating that there was no proof loaded. no_proof_header_still_visible

This is fixed now. no_proof_header_invisible

At "Proof"->"Show Active Taclet Options" you can see the active taclet options with a path given at the bottom to change the taclet options. active_taclet_settings_old

Now, the path to the taclet options has been changed to reflect the new way to change taclet options. active_taclet_settings_new

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

tobias-rnh avatar Feb 24 '24 09:02 tobias-rnh