Ticket #5694 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

MUON: Disable "Load Current Run" Button When Loading Files

Reported by: Peter Parker Owned by: Peter Parker
Priority: critical Milestone: Release 2.2
Component: Mantid Keywords:
Cc: anders.markvardsen@… Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Currently the "previous" and "next" arrow buttons of the Home tab of the Muon Analysis interface are disabled when they are pressed, for the duration of loading a run.

There is a problem where pressing "Load current run" and then the previous button immediately afterwards (or vice-versa) sometimes causes Mantid to load multiple runs one after the other in a continuous loop.

A good solution would be to disable "Load current run" (and possibly even the MWRunFiles widget) when loading runs with the interface.

Change History

comment:1 Changed 8 years ago by Peter Parker

Refs #5694 - Disable appropriate widgets on load.

Now the "Load current run" button and the MWRunFiles widget are disabled along with the previous and next arrow buttons whenever a run is loaded.

Changeset: 4dce34fda490fbd5c16581982edf35ad27073e1d

comment:2 Changed 8 years ago by Peter Parker

[Wrong ticket for comment].

Last edited 8 years ago by Peter Parker (previous) (diff)

comment:3 Changed 8 years ago by Peter Parker

  • Status changed from new to accepted

comment:4 Changed 8 years ago by Peter Parker

  • Status changed from accepted to verify
  • Resolution set to fixed

To test, make sure that the "Load current run" and arrow buttons work as they should, as well as the MWRunFiles widget.

Make sure pressing various combinations of these means that the buttons are disabled appropriately, and re-enabled when loading is finished. Make sure that trying to find or advance to runs that don't exist doesn't mean that the buttons are disabled and then left that way.

comment:5 Changed 8 years ago by Peter Parker

Refs #5694 - Remove accidentally added temp grouping file.

Basically just proving that I'm World Chamption at making simple jobs way way way harder than they have any right to be.

Changeset: d8526764f5f4150fac52a7e495764ad1b4f1863f

[This was meant for #5200 ...]

Last edited 8 years ago by Peter Parker (previous) (diff)

comment:6 Changed 8 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:7 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Works spot on

comment:8 Changed 8 years ago by Peter Parker

Refs #5694 - Disable appropriate widgets on load.

Now the "Load current run" button and the MWRunFiles widget are disabled along with the previous and next arrow buttons whenever a run is loaded.

Changeset: 4dce34fda490fbd5c16581982edf35ad27073e1d

comment:9 Changed 8 years ago by Peter Parker

Refs #5694 - Stop DBZ error.

Actually do something with the news that the user has typed in an incorrect value ...

Changeset: bdae8c6eccea736ed56daca3ac4460416a54cc30

comment:10 Changed 8 years ago by Peter Parker

Refs #5694 - Remove accidentally added temp grouping file.

Basically just proving that I'm World Chamption at making simple jobs way way way harder than they have any right to be.

Changeset: d8526764f5f4150fac52a7e495764ad1b4f1863f

comment:11 Changed 8 years ago by Peter Parker

Refs #5694 - Disable appropriate widgets on load.

Now the "Load current run" button and the MWRunFiles widget are disabled along with the previous and next arrow buttons whenever a run is loaded.

Changeset: 4dce34fda490fbd5c16581982edf35ad27073e1d

comment:12 Changed 8 years ago by Peter Parker

Refs #5694 - Stop DBZ error.

Actually do something with the news that the user has typed in an incorrect value ...

Changeset: bdae8c6eccea736ed56daca3ac4460416a54cc30

comment:13 Changed 8 years ago by Peter Parker

Refs #5694 - Remove accidentally added temp grouping file.

Basically just proving that I'm World Chamption at making simple jobs way way way harder than they have any right to be.

Changeset: d8526764f5f4150fac52a7e495764ad1b4f1863f

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6540

Note: See TracTickets for help on using tickets.