Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Datagrid: Issue with entering decimal number with Editors.Input using maskOptions #4974

Closed
claudenbach opened this issue Mar 19, 2021 · 9 comments · Fixed by #5209
Closed
Assignees
Labels
for: customer Issue came from a customer team: landmark For Landmark issues type: bug 🐛 [5] Velocity rating (Fibonacci)

Comments

@claudenbach
Copy link
Contributor

claudenbach commented Mar 19, 2021

Describe the bug
Attempting to enter a decimal value into a cell with Editors.Input and with maskOptions noticed...

  • if the cell is empty, entering '.25' will work
  • if the cell already has a value, entering '.25', you need to either enter the decimal point twice '..25' or enter '0.25'.

Steps to reproduce the behavior:

  1. Using the attached 'test-decimal-maskoptions.html'
  2. Set focus in Price cell of Row Id: 0 (has a value of 210.99)
  3. Enter '.25' => need to enter the decimal point '.' twice or enter '0.25'
  4. Set focus in the Price cell of Row Id: 3 (has no value)
  5. Enter '.25' => takes the value has entered (don't need enter the decimal point '.' twice of enter the leading zero

Expected Behavior
Entering a decimal value shouldn't need to enter the decimal point twice or enter leading 0.25

Version
v4.39.0-dev

@tmcconechy tmcconechy added the [5] Velocity rating (Fibonacci) label Mar 19, 2021
@tmcconechy
Copy link
Member

Probably an issue with mask. Css @EdwardCoyle

@claudenbach
Copy link
Contributor Author

Git-4974.zip

@claudenbach
Copy link
Contributor Author

Has this been planned for a sprint? The client associated with the LMCLIENT ticket is asking for updates.

@tmcconechy tmcconechy added this to To do in Enterprise 4.53.x (June 2021) Sprint via automation Apr 21, 2021
@tmcconechy
Copy link
Member

Put in June sprint for now..

@tmcconechy
Copy link
Member

@EdwardCoyle we will need to prioritize this one for landmark. Maybe it's easy (i hope). Probably have to make sure it works in the new code brought over.

@claudenbach
Copy link
Contributor Author

Seeing this same behavior on a numeric field that resides on a form.

  • if the field is empty, entering '.25' will work
  • however, if the field already has a value, entering '.25', you need to either enter the decimal point twice '..25' or enter '0.25'.

Form-Example

@tmcconechy
Copy link
Member

Yes that makes sense as i would expect this to be an issue in any mask...

@jbrcna
Copy link
Contributor

jbrcna commented May 25, 2021

the issue is now resolved. moving this ticket to Done.
v4.52.0-dev
https://main-enterprise.demo.design.infor.com/components/mask/test-number-leading-zeroes.html

@jbrcna jbrcna moved this from Ready for QA (beta) to Done in Enterprise 4.52.x (May 2021) Sprint May 25, 2021
@CindyMercadoReyes
Copy link
Collaborator

The BDD and accessibility test case has been created for this ticket. This component will be part of our regression test suite.
BDDStory_github4974.docx
TCs_Mask_LeadingZeroesinNumberMasks_V01.xlsx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for: customer Issue came from a customer team: landmark For Landmark issues type: bug 🐛 [5] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

6 participants