Ticket #10707 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

New start up screen doesn't appear on top of application on OS X

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: blocker Milestone: Release 3.3
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Wenduo Zhou

Description (last modified by Martyn Gigg) (diff)

To reproduce simply start the application on OS X and you will not see the new "first time" dialog but when you close MantidPlot you realise that it is behind the application.

Attachments

startup-dialog-osx10.9.png (329.4 KB) - added by Martyn Gigg 6 years ago.

Change History

comment:1 Changed 6 years ago by Martyn Gigg

  • Status changed from new to assigned
  • Description modified (diff)

comment:2 Changed 6 years ago by Nick Draper

  • Owner changed from Nick Draper to Martyn Gigg

Hmm. This will need some speculative development on the mac, so I was hoping you could help out.

I changed the dialog from being "Always on top" as is did not work with web links opening up underneath the "always on top" dialog.

This seems to be fine on windows and linux, but obviously not the mac. Perhaps that needs a conditional compilation part as long as Always on top does not cause the problem there. I don't know if the is a z order option to do a one off on top command.

comment:3 Changed 6 years ago by Martyn Gigg

There appear to be further issues on OS X 10.9. The dialog does not look good. See attached image.

Changed 6 years ago by Martyn Gigg

comment:4 Changed 6 years ago by Martyn Gigg

I also now do have a concern that it generally doesn't fit with the rest of the look and feel of MantidPlot.

Could we just add the links to the old dialog? While basic, it was in keeping with everything else.

comment:5 Changed 6 years ago by Martyn Gigg

  • Status changed from assigned to inprogress

Formatting changes only.

Refs #10707

Changeset: 129f52adf30a84e0d87effeef279076b22d61770

comment:6 Changed 6 years ago by Martyn Gigg

Delay raising first time dialog until event loop is processing.

If not, the dialog appers below everything on OS X as raise() does nothing Refs #10707

Changeset: 7669e84fdfc4be32cec69cb8f22aec4688a53582

comment:7 Changed 6 years ago by Nick Draper

re #10707 attempt at better scalaing using px in stylesheets

Changeset: 5e80e0801f8fb22d87d55f528e5ef570af72a45c

comment:8 Changed 6 years ago by Martyn Gigg

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

Branch: bugfix/10707_startup_dialog_z_order

Tester: Take the Mac package from http://builds.mantidproject.org/job/develop_clean/label=osx-10.8-build/. Install it and start MantidPlot. The dialog should now appear on top.

comment:9 Changed 6 years ago by Wenduo Zhou

  • Status changed from verify to verifying
  • Tester set to Wenduo Zhou

comment:10 Changed 6 years ago by Martyn Gigg

Merge branch 'master' into bugfix/10707_startup_dialog_z_order

Conflicts:

Code/Mantid/MantidPlot/src/Mantid/FirstTimeSetup.ui

Refs #10707

Changeset: a514c5d2e276fa9aa1ada0743e5275d643f62170

comment:11 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge branch 'master' into bugfix/10707_startup_dialog_z_order

Conflicts:

Code/Mantid/MantidPlot/src/Mantid/FirstTimeSetup.ui

Refs #10707

Full changeset: a514c5d2e276fa9aa1ada0743e5275d643f62170

comment:12 Changed 6 years ago by Wenduo Zhou

Merge remote-tracking branch 'origin/bugfix/10707_startup_dialog_z_order'

Full changeset: 0f8ea426e92d61133fd054916fa4da3b5eec0099

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11549

Note: See TracTickets for help on using tickets.