Ticket #4088 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

ColorBarWidget: do not limit user's entry for min/max range

Reported by: Owen Arnold Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.0
Component: VATES Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

If you modify the minimum on the colour scale in the slice viewer, it immediately replaces this value with it's default. Modifications are remembered for maximum, but not for the minimum.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

I have the color bar widget limit the user's entry of min/max to the min/max of the full data range calculated when setting the workspace. That means that if you set min < overall minimum, it will reset. If you set it larger than that, it will stick.

I'm thinking that maybe I should NOT limit the min/max this way but allow the user to put whatever. What do you think?

comment:2 Changed 9 years ago by Owen Arnold

I think that the users should be able to set it lower than the overall range. That way they have more control over the scaling.

comment:3 Changed 9 years ago by Janik Zikovsky

  • Summary changed from Slice viewer forgets minimum to Slice viewer: do not limit user's entry for min/max range

Modified ticket title...

comment:4 Changed 9 years ago by Janik Zikovsky

  • Summary changed from Slice viewer: do not limit user's entry for min/max range to ColorBarWidget: do not limit user's entry for min/max range

comment:5 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:6 Changed 9 years ago by Janik Zikovsky

Refs #4088 : Color map not limited

It seems to act funny if min < 1e-8 but I think this is an issue with QwtPlot, not my code.

Changeset: f76d2e4c086cf387a074fcce7b6bede2a76ba950

comment:7 Changed 9 years ago by Janik Zikovsky

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

comment:8 Changed 9 years ago by Janik Zikovsky

Note to testers: This widget is used inside the SliceViewer.

comment:9 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:10 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

This works. Defaults to the range of real values, but can specify these boundaries to be outside the pre-set limits.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4935

Note: See TracTickets for help on using tickets.