Ticket #5934 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Instrument Options on First Time Setup

Reported by: Owen Arnold Owned by: Russell Taylor
Priority: critical Milestone: Release 2.6
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Gesner Passos

Description (last modified by Nick Draper) (diff)

See Test listener in First Time Setup Dialog.

It makes the Mantid project look sloppy since this is the first GUI component first-time-users can interact with, and it contains options that are wrong/invalid. That's why I've made this ticket critical to fix.

Attachments

instrument_options.png (37.8 KB) - added by Owen Arnold 8 years ago.

Change History

Changed 8 years ago by Owen Arnold

comment:1 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:2 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.4 to Release 2.5

Moved at the code freeze for release 2.4

comment:3 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:4 Changed 7 years ago by Russell Taylor

  • Owner changed from Anyone to Russell Taylor
  • Status changed from new to accepted

My contribution to reducing the number of critical bugs in 'Anyone'.

comment:5 Changed 7 years ago by Russell Taylor

Re #5934. Move the TESTHISTOLISTENER instrument to the TEST facility.

This stops it appearing in the list of ISIS instruments in the first-time setup dialog and elsewhere.

Changeset: ae21c8ba4261e8c2ad7b479cfbe6eec3440d89f5

comment:6 Changed 7 years ago by Russell Taylor

Re #5934. Fix python FacilityInfo test for 1 fewer ISIS instruments.

Changeset: bd26063b56e0aa48f6485fa852cbcaa1494f623a

comment:7 Changed 7 years ago by Russell Taylor

Re #5934. Fix python FacilityInfo test for 1 fewer ISIS instruments.

Changeset: bd26063b56e0aa48f6485fa852cbcaa1494f623a

comment:8 Changed 7 years ago by Russell Taylor

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

Branch is bugfix/5934_hide_testhistolistener

To test:

  • Delete the default.facility & default.instrument lines from your Mantid.user.properties file
  • Start MantidPlot and note that the 'TESTHISTOLISTENER' is absent from the ISIS list of instruments, unlike it was in the attached figure
  • It also used to be present in other places that listed the ISIS instruments, such as the StartLiveData GUI (when ISIS is the default instrument) and probably the ICat dialogs as well.

comment:9 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:10 Changed 7 years ago by Russell Taylor

Re #5934. Move the TESTHISTOLISTENER instrument to the TEST facility.

This stops it appearing in the list of ISIS instruments in the first-time setup dialog and elsewhere.

Changeset: ae21c8ba4261e8c2ad7b479cfbe6eec3440d89f5

comment:11 Changed 7 years ago by Russell Taylor

Re #5934. Fix python FacilityInfo test for 1 fewer ISIS instruments.

Changeset: bd26063b56e0aa48f6485fa852cbcaa1494f623a

comment:12 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to closed

No more TESTHISTOLISTENER for new user, neither for first setup from the menu.

fixed.

comment:13 Changed 7 years ago by Russell Taylor

Re #5934. Move the TESTHISTOLISTENER instrument to the TEST facility.

This stops it appearing in the list of ISIS instruments in the first-time setup dialog and elsewhere.

Changeset: ae21c8ba4261e8c2ad7b479cfbe6eec3440d89f5

comment:14 Changed 7 years ago by Russell Taylor

Re #5934. Fix python FacilityInfo test for 1 fewer ISIS instruments.

Changeset: bd26063b56e0aa48f6485fa852cbcaa1494f623a

comment:15 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:16 Changed 7 years ago by Nick Draper

  • Component changed from Framework to User Interface
  • Description modified (diff)

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6780

Note: See TracTickets for help on using tickets.