Ticket #8380 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

[ICAT] Centre GUI and calendar

Reported by: Jay Rainey Owned by: Jay Rainey
Priority: minor Milestone: Release 3.1
Component: GUI Keywords: ICAT, ICAT4
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description (last modified by Jay Rainey) (diff)

When the ICAT GUI is opened (or the related ICAT calendar) they appear to be placed in random positions on the screen. Ideally, they should be positioned in the center of the screen.

Change History

comment:1 Changed 7 years ago by Jay Rainey

  • Description modified (diff)

comment:2 Changed 7 years ago by Jay Rainey

  • Blocked By 8244 added

When testing #8322 Russel noted:

  • The search box appears in strange places (like the top-left of the screen). It should appear in the centre of the MantidPlot window. If I recall correctly, the way to do this is to make the ApplicationWindow the parent of the widget. This will also give the window the Mantid icon.
Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:3 Changed 7 years ago by Jay Rainey

  • Status changed from new to inprogress

In Mantid 2.6 the ICAT GUI appeared in the center of the MantidPlot window. However, this prevents the user from dragging the GUI to another desktop. Moreover, the new ICAT GUI can become quite long when there as many results returned, which is not pleasant to view when the interface is within MantidPlot.

Instead, I have opted to center the current GUI (& calendar) to the middle of the screen and keep the ICAT GUI in it's own window.

Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:4 Changed 7 years ago by Jay Rainey

Centre ICAT GUI on screen. Refs #8380.

Changeset: 19026f612d7407ce48a921b852a9df8393c47dce

comment:5 Changed 7 years ago by Jay Rainey

Centre ICAT calendar on screen. Refs #8380.

Changeset: 19678c77574a4cde09feb94143d4c4b12ccef80a

comment:6 Changed 7 years ago by Jay Rainey

  • Blocked By 8244 removed

To test

  1. Open Mantid & log into the catalog.
  2. Click Search from the Catalog menu. The ICAT GUI should now be centered on screen.
  3. Click the calendar button ([...]) to open a calendar. The calendar should now be centered on screen.

Note: Issues related to the calendar have been noted and addressed in #8244.

comment:7 Changed 7 years ago by Jay Rainey

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

comment:8 Changed 7 years ago by Samuel Jackson

  • Status changed from verify to verifying
  • Tester set to Samuel Jackson

comment:9 Changed 7 years ago by Samuel Jackson

  • Status changed from verifying to reopened
  • Resolution fixed deleted

The calendar isn't actually created at the centre of the screen. I would suggest resizing the calendar first, then centring it as discussed.

comment:10 Changed 7 years ago by Jay Rainey

  • Status changed from reopened to inprogress

Set the height when aligning calendar. Refs #8380.

Changeset: d5c197904f0aea3934a52fbcdd4441fcd50106ee

comment:11 Changed 7 years ago by Jay Rainey

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

The calendar should now center correctly.

comment:12 Changed 7 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester changed from Samuel Jackson to Owen Arnold

comment:13 Changed 7 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8380_icat_centre_gui'

Full changeset: ​fd82e3ab2d954f9eb7a9952c7c271eb9eb92cddb

comment:14 Changed 7 years ago by Owen Arnold

Centering is working. Functionality seems unaffected otherwise. No errors or warnings. Code review looks good.

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9225

Note: See TracTickets for help on using tickets.