Ticket #4893 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

NormaliseToMonitor: auto generated GUI

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: alex.buts@… Blocked By:
Blocking: Tester: Russell Taylor

Description

  • Load workspace with monitors
  • Select the workspace a few times
  • MonitorID PropertyWidget does not get deleted properly.
  • On linux they get overlaid
  • On windows they are next to each other

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

I believe this has been fixed by another commit (removing a insertChild() call in PropertyWidget). Tester: Please confirm on Windows.

comment:3 Changed 9 years ago by Janik Zikovsky

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

comment:4 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:5 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to reopened
  • Resolution fixed deleted

It looks OK on Linux & Windows, but there's still a problem on the Mac. The widgets appear to be overlaid and making a selection from the drop-down box can crash Mantid.

A short stack trace:

0 QtGui QWidget::setEnabled(bool) + 19
1 libMantidQtAPI.dylib MantidQt::API::PropertyWidget::setEnabled(bool) + 494

Tested in 2.0.2345-g1a0f8f5

Last edited 8 years ago by Russell Taylor (previous) (diff)

comment:6 Changed 8 years ago by Russell Taylor

I just got it to crash on Windows as well. It looks like things are being overlaid there as well (look at the property name - the font looks a bit different to the others - as if it's bold).

comment:7 Changed 8 years ago by Janik Zikovsky

  • Status changed from reopened to accepted

comment:8 Changed 8 years ago by Janik Zikovsky

Refs #4893: hope this properly fixes NormaliseToMonitor GUI

Changeset: 720b5d587a5764a6a65ca599555b7b77231e497e

comment:9 Changed 8 years ago by Janik Zikovsky

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

Seems OK on my machine but it'd be best to try on Windows...

comment:10 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying

comment:11 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

Looks OK now. Couldn't get it to crash on Mac or Windows.

comment:12 Changed 8 years ago by Janik Zikovsky

Refs #4893: hope this properly fixes NormaliseToMonitor GUI

Changeset: 720b5d587a5764a6a65ca599555b7b77231e497e

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5739

Note: See TracTickets for help on using tickets.