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: Update NumberBox formatting pattern for precision #1080

Merged
merged 6 commits into from
Jul 9, 2024

Conversation

FourLeafTec
Copy link
Contributor

@FourLeafTec FourLeafTec commented Jul 2, 2024

fix #1079 .
By add NumberFormat pattern L632.

@bdlukaa
Copy link
Owner

bdlukaa commented Jul 3, 2024

Great! Would you mind adding this to the CHANGELOG?

@FourLeafTec
Copy link
Contributor Author

Sure, but I just realized that this PR is a bit too simple, and I would like to add some more content:

  1. Create properties to store the pattern instead of concatenating them each time
  2. Allow users to pass in patterns to customize the format
  3. Allow a format method to customize the formatting, but this will affect the parser, and I haven't figured this out yet. I would like to hear your thoughts.

It seems that the PR has also failed to merge, so I will find some time later to complete this part and then modify the CHANGLOG.. Is that okay?

@bdlukaa
Copy link
Owner

bdlukaa commented Jul 4, 2024

Sounds okay to me!

@bdlukaa
Copy link
Owner

bdlukaa commented Jul 8, 2024

@FourLeafTec let me known when this is ready for review!

@FourLeafTec
Copy link
Contributor Author

@bdlukaa It‘s ok now, and feedback is welcome.

lib/src/controls/form/number_box.dart Outdated Show resolved Hide resolved
lib/src/controls/form/number_box.dart Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
FourLeafTec and others added 3 commits July 9, 2024 00:28
Co-authored-by: Bruno D'Luka <45696119+bdlukaa@users.noreply.github.com>
Co-authored-by: Bruno D'Luka <45696119+bdlukaa@users.noreply.github.com>
@FourLeafTec
Copy link
Contributor Author

@bdlukaa Review please.

@bdlukaa bdlukaa merged commit 0b71276 into bdlukaa:master Jul 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Numberbox format notwork when precision bigger then 4
2 participants