Ticket #4271 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SliceViewer: put the menu in the QMainWindow

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: minor Milestone: Release 2.0
Component: MantidPlot Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description

I had the SliceViewerWidget have its own menu bar, but that looks funny now that its container is a QMainWindow.

Move all its menus to the parent QMainWindow if one is found. Look automatically for the parent window in SliceViewer constructor.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Janik Zikovsky

Refs #4271: menu looks better in SliceViewer

Changeset: 1a84d522a3f1b33ae7bb8913bceb46f3bf3deb77

comment:3 Changed 9 years ago by Janik Zikovsky

Refs #4272: Handy object that links a menu and a checkable button

Also refs #4271: used that to sync a menu in the SliceViewer with the button.

Changeset: 174031223f0971e19550a1ccec79fe71bad21ffb

comment:4 Changed 9 years ago by Janik Zikovsky

Refs #4271: forgot git add!

Changeset: e44eecdd24469e965aca10c8261f729c47af1b16

comment:5 Changed 9 years ago by Janik Zikovsky

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

comment:6 Changed 9 years ago by Janik Zikovsky

Refs #4271: menu looks better in SliceViewer

Changeset: 1a84d522a3f1b33ae7bb8913bceb46f3bf3deb77

comment:7 Changed 9 years ago by Janik Zikovsky

Refs #4272: Handy object that links a menu and a checkable button

Also refs #4271: used that to sync a menu in the SliceViewer with the button.

Changeset: 174031223f0971e19550a1ccec79fe71bad21ffb

comment:8 Changed 9 years ago by Janik Zikovsky

Refs #4271: forgot git add!

Changeset: e44eecdd24469e965aca10c8261f729c47af1b16

comment:9 Changed 9 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:10 Changed 9 years ago by Andrei Savici

  • Status changed from verifying to closed

Looks fine to me

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5118

Note: See TracTickets for help on using tickets.