[GitHub] Harbs commented on issue #40: NumericStepper can exceed maximum value

[GitBox <gi...@apache.org>]

Harbs commented on issue #40: NumericStepper can exceed maximum value
URL: https://github.com/apache/royale-asjs/issues/40#issuecomment-339909617
 
 
   There is a conflict between the maximum and the snapInterval. If the snapInterval is 7, the maximum has to be a multiple of 7 for the maximum to make sense.
   
   When there's a conflict, there's three possible ways to resolve it:
   1. maximum is the maximum value *before* the snap (as it behaves now)
   2. maximum is not really the maximum. Rather it's the nearest snap value to the maximum.
   3. The snapInterval is ignored at the maximum value.
   
   None of these are truly correct answers and I don't think it't the job of the RangeModel to make these decisions. The RangeModel implementation should be the implementation which has the smallest overhead.
   
   I think we should leave it as-is. If someone needs a more sophisticated RangeModel, it should be implemented according to the needs of the client, and specified in the CSS of the app.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services