Ticket #1958 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Plot controls disabled

Reported by: Peter Peterson Owned by: Roman Tolchenov
Priority: critical Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description

The plot controls (like zooming) are completely disabled by default. To reproduce the bug:

  1. Load data
  2. Right click on a workspace and plot specturm
  3. Zoom controls don't work

At one point I was able to get around it:

  1. In the menus View->Preferences select Mantid
  2. Click the apply button
  3. Zoom controls are enabled again

Attachments

Mantid1958.png (45.7 KB) - added by Janik Zikovsky 10 years ago.
MDI bug in Mantid
Screenshot-1.png (53.3 KB) - added by Janik Zikovsky 10 years ago.

Change History

comment:1 Changed 10 years ago by Russell Taylor

I don't have this problem (checked on red hat, mac & windows). Ubuntu specific?

comment:2 Changed 10 years ago by Nick Draper

  • Owner changed from Roman Tolchenov to Martyn Gigg
  • Status changed from new to assigned

Can you just check this on ubuntu, then we'll work out what to do.

comment:3 Changed 10 years ago by Martyn Gigg

I've had a play and I can't seem to reproduce it either. Time for a check of the code possibly to see when the controls get enabled/disabled to see if there is anything obvious?

comment:4 Changed 10 years ago by Peter Peterson

This is an intermittent problem. It was fixed (I assumed that somebody actively did something), then broke, and now works again over the course of the two weeks it has been open.

comment:5 Changed 10 years ago by Janik Zikovsky

I can confirm the same bug on my system: Ubuntu 10.10. Opening one of the Interfaces menu fixes it.

comment:6 Changed 10 years ago by Janik Zikovsky

  • Status changed from assigned to accepted
  • Owner changed from Martyn Gigg to Janik Zikovsky

comment:7 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

Changed 10 years ago by Janik Zikovsky

MDI bug in Mantid

comment:8 Changed 10 years ago by Janik Zikovsky

  • Owner changed from Janik Zikovsky to Roman Tolchenov
  • Status changed from accepted to assigned

Did a lot of debugging but could not fix it. It looks like there is some bug in the way the MDI area is acting. In some cases, you end up with two MDI windows simultaneously having focus. Then neither window receives the WindowActivated signal it needs in order to activate the tools.. See my attached screenshot.

comment:9 Changed 10 years ago by Roman Tolchenov

Cannot reproduce the bugs.

comment:10 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:11 Changed 10 years ago by Roman Tolchenov

  • Status changed from assigned to accepted

comment:12 Changed 10 years ago by Roman Tolchenov

(In [9770]) Added some magical code which seems to do the job of getting QMdiArea out of its confused state. re #1958

comment:13 Changed 10 years ago by Roman Tolchenov

(In [9775]) Fixed a similar issue when Matrix Plot toolbar is not activated when a MantidMatrix gets focus. re #1958

Changed 10 years ago by Janik Zikovsky

comment:14 Changed 10 years ago by Janik Zikovsky

Hi Roman,

I just tried your changes and it looks like that fixed it, thanks a lot (that was an annoying bug)!

One thing I noticed though is that if you have no windows open, and right-click on a workspace and click "plot spectrum", the window that opens is not in the same document area as the other ones (it can go on top of toolbars), and the zoom/etc. buttons are disabled. I've attached a screenshot.

Thanks,

Janik

comment:15 Changed 10 years ago by Roman Tolchenov

(In [9796]) This should fix the introduced bug. re #1958

comment:16 Changed 10 years ago by Roman Tolchenov

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

comment:17 Changed 9 years ago by Andrei Savici

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

comment:18 Changed 9 years ago by Andrei Savici

  • Status changed from verifying to closed

Works on Ubuntu 11.04beta

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2805

Note: See TracTickets for help on using tickets.