Wrong prices displayed: missing values after "." about pricing-page

[xiahengchen]

Hi @Dudo1985, thank you for reporting! We have fixed the issue that you reported and so you can checkout feature/pricing-amount-fraction-fix branch to check the changes.

[iksent]

Hello, @xiahengchen!
I have just checked pricing-amount-fraction-fix and it's not working as expected: there are no dots for now, but the prices aren't correct, take a look:

Prev version:

And the new version:

PS: And may be we should use dot instead of comma at "Normally ..." text?

[iksent]

By the way, don't you think, that we should display price per year, not per month, so users can see the final price instantly without calculations in their heads ;)

Or at least display per month and per year.

[vovafeldman]

@iksent the code is intentionally using num.toLocaleString() to show the prices based on the local numeric format. Some countries use "." for decimal digits separation, but some use ",". That's btw also the reason we failed to fix the bug on the first try. @xiahengchen is already working on the fix.

With regard to your question, it's psychologically easier to commit when seeing smaller numbers rather than showing the annual price, so this practice is known to increase conversion rate. At a later stage, we may introduce a flag for you to choose if you prefer to show the annual prices.

[iksent]

+1 for flag and more configurations :)

[xiahengchen]

@iksent Please pull the latest changes of feature/pricing-amount-fraction-fix branch and you can see the bug has been fixed. Please let me know if this works for you.

[iksent]

Hello, @xiahengchen!

There is the result of the latest fixes:

I have a special JS-function to display the prices correctly. You can try to use it too:

export const toFixedN = (
  number: ?number,
  fractionDigits: number = 4,
  canBeNull: boolean = true,
): string => {
  let num = number

  num = +num
  if (isNaN(num)) {
    return '-'
  }
  if (num === 0) {
    return canBeNull ? '0' : '~0'
  }
  if (num < 1) {
    num = num.toFixed(fractionDigits).replace(/\.?0+$/, '')
  } else {
    num = parseFloat(num.toFixed(fractionDigits))
  }
  return +num === 0 ? `~${num}` : num.toString()
}

This function works like standard toFixed but also handles small values (<1) correctly.

[xiahengchen]

Sorry for the inconvenience @iksent. We fixed the above issue and pushed the changes to the feature/pricing-amount-fraction-fix branch. Please check it and let us know if you still see any issues.

[vovafeldman]

Merged to the develop branch PR #9