Ticket #7049 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Mantidplot central area doesn't redraw after waterfall plot resize

Reported by: Roman Tolchenov Owned by: Russell Taylor
Priority: major Milestone: Release 2.6
Component: MantidPlot Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description


Attachments

waterfall_plot_resize.png (96.0 KB) - added by Roman Tolchenov 7 years ago.

Change History

Changed 7 years ago by Roman Tolchenov

comment:1 Changed 7 years ago by Roman Tolchenov

It usually happens when a qt event is triggered inside an event handler

comment:2 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Russell Taylor

comment:3 Changed 7 years ago by Russell Taylor

  • Status changed from assigned to accepted

The guilty party is the replot() call inside Graph::updateDataCurves()

comment:4 Changed 7 years ago by Russell Taylor

Re #7049. Remove replot that affected refresh on window resizing.

When resizing a docked waterfall plot, the background was not being refreshed properly. The issue was traced to this call, which is not needed in this context.

Changeset: 838e04eb8bb022072b3bbaa7f119f158eec2e350

comment:5 Changed 7 years ago by Russell Taylor

Re #7049. Add calls to replot where now required...

...after a call to updateDataCurves().

Changeset: 1744bb975c0eec6a1700e502e17c220544d22033

comment:6 Changed 7 years ago by Russell Taylor

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

Tester: Create a waterfall plot & test resizing it (see the attached figure for the previous behaviour which was present at least on windows & linux). Also do some exploratory testing of other aspects of waterfall plots: reversing the order, changing the offset, adding/removing curves etc. In all these cases the graph should be responsive in updating.

Branch is feature/7049_waterfall_resizing

comment:7 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Checked this on Windows, RHEL6 and Ubuntu. Seems to be working fine.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7895

Note: See TracTickets for help on using tickets.