Negative numbers with .addNumericEntry #81

Closed
jacobthetechy opened this Issue Jan 4, 2017 · 5 comments

Comments

Projects
None yet
2 participants
@jacobthetechy
Contributor

jacobthetechy commented Jan 4, 2017

When an entry is .addNumericEntry you can not use an negative or a period before typing in an number. What you have to do it type your number then either use the arrow keys or click at the beginning of the entry form and then add "-" or "." .

@jarvisteach jarvisteach self-assigned this Jan 4, 2017

@jarvisteach jarvisteach added the bug label Jan 4, 2017

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Jan 4, 2017

Owner

I've had a quick look - the characters are accepted in the character validation, but it then tries to cast the contents of the entry as a float every time you type a character.

Having just a minus or decimal point, or both throws an exception as they're not valid floats.

However, you're right they should be allowed - I'll add in specific use cases to allow those three combinations.

Owner

jarvisteach commented Jan 4, 2017

I've had a quick look - the characters are accepted in the character validation, but it then tries to cast the contents of the entry as a float every time you type a character.

Having just a minus or decimal point, or both throws an exception as they're not valid floats.

However, you're right they should be allowed - I'll add in specific use cases to allow those three combinations.

jarvisteach added a commit that referenced this issue Jan 5, 2017

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Jan 5, 2017

Owner

Will be fixed in next release

Owner

jarvisteach commented Jan 5, 2017

Will be fixed in next release

@jarvisteach jarvisteach closed this Jan 5, 2017

@jacobthetechy

This comment has been minimized.

Show comment
Hide comment
@jacobthetechy

jacobthetechy Jan 5, 2017

Contributor

What is the next release, if you don't mind me asking?

Contributor

jacobthetechy commented Jan 5, 2017

What is the next release, if you don't mind me asking?

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Jan 6, 2017

Owner

I'll hopefully be able to push one out over the weekend.

Owner

jarvisteach commented Jan 6, 2017

I'll hopefully be able to push one out over the weekend.

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Jan 8, 2017

Owner

New version released.

Owner

jarvisteach commented Jan 8, 2017

New version released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment