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: change number formatting with comma as decimal separator #6655

Merged
merged 8 commits into from
Oct 2, 2024

Conversation

DiegoCardoso
Copy link
Contributor

@DiegoCardoso DiegoCardoso commented Sep 18, 2024

Description

Change the number formatting configuration for locales that use commas as decimal separators. Before, the fraction part of the numbers in such locales would allow only 3 digits. This change allows more digits in the fractional part.

Fixes #5731

Type of change

  • Bugfix
  • Feature

@DiegoCardoso DiegoCardoso changed the title wip: format numbers in formula fix: change number formatting with comma as decimal separator Sep 24, 2024
@DiegoCardoso DiegoCardoso marked this pull request as ready for review September 24, 2024 08:38
Copy link

sonarcloud bot commented Oct 2, 2024

@DiegoCardoso DiegoCardoso merged commit 1c0c5b2 into main Oct 2, 2024
5 checks passed
@DiegoCardoso DiegoCardoso deleted the fix/spreadsheet/formula-number-format branch October 2, 2024 08:42
@vaadin-bot
Copy link
Collaborator

Hi @DiegoCardoso and @DiegoCardoso, when i performed cherry-pick to this commit to 24.4, i have encountered the following issue. Can you take a look and pick it manually?
Error Message:
Error: Command failed: git cherry-pick 1c0c5b2
error: could not apply 1c0c5b2... fix: change number formatting with comma as decimal separator (#6655)
hint: After resolving the conflicts, mark them with
hint: "git add/rm ", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.5.0.beta5 and is also targeting the upcoming stable 24.5.0 version.

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

Successfully merging this pull request may close these issues.

Spreadsheet: Incorrect number formatting in formula - locale and precision is lost
3 participants