MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0006127ParaView(No Category)public2007-12-05 11:562009-05-13 13:59
ReporterDavid Karelitz 
Assigned ToClinton Stimpson 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.4Fixed in Version3.4 
Project
Topic Name
Type
Summary0006127: (MARS-low) Maximum value in Set Scalar Range is checked against the minimum value on every keystroke
DescriptionIf you go to the Scalar Bar window and hit Set Scalar Range, the maximum value box is checked against the minimum value box on every keypress. This makes entering a value such as 3e7 impossible when the minimum value is 4000. After hitting the 3, the value changes to 4000.

The maximum value should only be verified when return is pressed or the box loses focus.

The workaround is to enter a dummy minimum value, enter the maximum value, then enter the correct minimum value
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
has duplicate 0006270closed Ken Moreland ParaView checks ranges every keystroke for color editor 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2007-12-05 11:56David KarelitzNew Issue
2008-03-07 07:56Berk GeveciTarget Version => MARS
2008-03-07 08:03Berk GeveciCategoryDevelopment => 3.4
2008-04-08 15:35Ken MorelandStatusbacklog => @80@
2008-04-08 15:35Ken MorelandResolutionopen => fixed
2008-04-08 15:35Ken MorelandAssigned To => Ken Moreland
2008-04-08 15:35Ken MorelandNote Added: 0011172
2008-04-08 18:40Alan ScottStatus@80@ => @20@
2008-04-08 18:40Alan ScottResolutionfixed => reopened
2008-04-08 18:40Alan ScottNote Added: 0011195
2008-04-09 17:08Ken MorelandNote Added: 0011228
2008-04-09 17:09Ken MorelandStatus@20@ => tabled
2008-04-09 17:09Ken MorelandAssigned ToKen Moreland => Berk Geveci
2008-04-09 18:09Ken MorelandPrioritylow => high
2008-04-09 18:09Ken MorelandRelationship addedhas duplicate 0006270
2008-04-14 19:11Alan ScottNote Added: 0011305
2008-05-22 13:47Berk GeveciAssigned ToBerk Geveci => Clinton Stimpson
2008-05-23 12:55Clinton StimpsonStatustabled => @80@
2008-05-23 12:55Clinton StimpsonResolutionreopened => fixed
2008-05-23 12:55Clinton StimpsonNote Added: 0012094
2008-06-02 20:04Alan ScottStatus@80@ => closed
2008-06-02 20:04Alan ScottNote Added: 0012205
2009-05-13 13:58Utkarsh AyachitTarget VersionMARS => 3.4
2009-05-13 13:59Utkarsh AyachitFixed in Version => 3.4
2011-06-16 13:10Zack GalbreathCategory => (No Category)

Notes
(0011172)
Ken Moreland   
2008-04-08 15:35   
Someone (not me) appears to have fixed this problem. There is now a short delay between a keystroke and the range check. It is now really easy to type (in the example above) 3e7, unless you're a really slow typist.
(0011195)
Alan Scott   
2008-04-08 18:40   
This is not an acceptable solution. Use cases that need to be solved are as follows:
* Slow typist. Minimum and maximum should be changeable at any speed.
* Minimum can be changed above maximum. Change range of 0-1 to 100-1000, starting with the minimum. Do same, changing range from 100-1000 to 0-1, starting with maximum.
* Field can go to blank, THEN be filled in. Delete maximum. Wait 3 seconds. Type in a number.

Suggestion: only range check when user selects Rescale. Bring up error dialog box, or change the colors of the input and don't close the input dialog, when there is an error. Don't try to guess what the user wants and change things. This is frustrating at best, error prone at worst.
(0011228)
Ken Moreland   
2008-04-09 17:08   
I like the previous suggestion, but with a slight twist. While the max is less than the min, the text should change to bold red and the rescale button should be grayed out. This both makes it clear what the problem is and makes it impossible to select an invalid range.
(0011305)
Alan Scott   
2008-04-14 19:11   
I agree with Ken.
(0012094)
Clinton Stimpson   
2008-05-23 12:55   
Fixed/cleaned up a few other things too.

/cvsroot/ParaView3/ParaView3/Qt/Components/pqColorScaleEditor.cxx,v <-- Components/pqColorScaleEditor.cxx
new revision: 1.44; previous revision: 1.43
/cvsroot/ParaView3/ParaView3/Qt/Components/pqRescaleRange.cxx,v <-- Components/pqRescaleRange.cxx
new revision: 1.3; previous revision: 1.2
/cvsroot/ParaView3/ParaView3/Qt/Components/pqRescaleRange.h,v <-- Components/pqRescaleRange.h
new revision: 1.3; previous revision: 1.2
/cvsroot/ParaView3/ParaView3/Qt/Widgets/CMakeLists.txt,v <-- Widgets/CMakeLists.txt
new revision: 1.33; previous revision: 1.32
/cvsroot/ParaView3/ParaView3/Qt/Widgets/pqLineEditNumberValidator.cxx,v <-- Widgets/pqLineEditNumberValidator.cxx
new revision: delete; previous revision: 1.2
/cvsroot/ParaView3/ParaView3/Qt/Widgets/pqLineEditNumberValidator.h,v <-- Widgets/pqLineEditNumberValidator.h
new revision: delete; previous revision: 1.2
(0012205)
Alan Scott   
2008-06-02 20:04   
Nice.
Tested client/server.