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

fix(InputMasked): fix integerLimit to work during typing #1694

Merged
merged 1 commit into from
Nov 6, 2022

Conversation

tujoworker
Copy link
Member

@tujoworker tujoworker commented Nov 3, 2022

Here is reprod of the bug.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 3, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8993e19:

Sandbox Source
eufemia-starter Configuration
eufemia-starter (forked) PR

@gatsby-cloud
Copy link

gatsby-cloud bot commented Nov 3, 2022

✅ DNB Eufemia Portal deploy preview ready

@tujoworker tujoworker marked this pull request as ready for review November 3, 2022 20:51
Copy link
Contributor

@langz langz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jorgenholme
Copy link

jorgenholme commented Nov 4, 2022

It looks like the limit stops at 8 integers even though the limit is set to 10. Is it counting the spaces maybe?
image

@tujoworker
Copy link
Member Author

yes, good point. Will fix that 👍

@tujoworker tujoworker force-pushed the fix/input-masked-limit branch from 9ac75fb to 8993e19 Compare November 4, 2022 11:08
@tujoworker
Copy link
Member Author

@jorgenholme Can you check if the latest changes work as expected? Here are the changes.

@jorgenholme
Copy link

Looks great 💯

@tujoworker tujoworker merged commit 0fb697d into main Nov 6, 2022
@tujoworker tujoworker deleted the fix/input-masked-limit branch November 6, 2022 19:27
tujoworker pushed a commit that referenced this pull request Nov 7, 2022
# [9.37.0](v9.36.0...v9.37.0) (2022-11-07)

### Bug Fixes

* **InputMasked:** fix integerLimit to work during typing ([#1694](#1694)) ([0fb697d](0fb697d))
* **Table:** remove usage of pseudo elements inside table rows ([#1704](#1704)) ([3f678bd](3f678bd))
* **Td:** fix children and add tests ([#1703](#1703)) ([060e84f](060e84f))

### Features

* add e_scooter icon ([#1701](#1701)) ([c7024e2](c7024e2))
@tujoworker
Copy link
Member Author

🎉 This PR is included in version 9.37.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants