Ticket #2882 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

ITableWorkspace: Allow editing in MantidPlot when appropriate

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: critical Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

This is needed by PeaksWorkspace. A mechanism for linking back the GUI table values and changing the underlying values needs to exist.

Other TableWorkspaces should probably be read-only by default. Setting columns read-only should be made smart, because not all PeakWorkspace values should be edited.

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

  • Summary changed from ITableWorkspace: Allow editing in MantidPlot when appropraite to ITableWorkspace: Allow editing in MantidPlot when appropriate

comment:3 Changed 9 years ago by Janik Zikovsky

(In [12137]) Refs #2882: Indentation fix only

comment:4 Changed 9 years ago by Janik Zikovsky

(In [12138]) Refs #2882: Clean up of MantidTable, reduced code duplication.

comment:5 Changed 9 years ago by Janik Zikovsky

(In [12143]) Refs #2882: Can manually edit some of the columns of a PeaksWorkspace directly in MantidPlot.

comment:6 Changed 9 years ago by Janik Zikovsky

  • Milestone changed from Iteration 29 to Iteration 30

Moved my tickets that won't get finished by iteration 29

comment:7 Changed 9 years ago by Janik Zikovsky

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

This has been working in PeaksWorkspaces for a while now, and I see no reason to make a general mechanism for TableWorkspace since I've seen no need for it. Fixed.

For testers: check that editing values in the Peaks table modifies the underlying data.

comment:8 Changed 9 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:9 Changed 9 years ago by Martyn Gigg

  • Status changed from verifying to closed

Verified by creating peaks table, editing some values, closing the table and reloading it. Confirmed that the requested changes had been made. Also confirmed that some only certain columns can be edited.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3729

Note: See TracTickets for help on using tickets.