Skip to content

Conversation

@SupermanBritt
Copy link
Contributor

No description provided.

Copy link
Member

@eschenfeldt eschenfeldt left a comment

Choose a reason for hiding this comment

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

I think we'll also need to add Percent to the supported control lists for the options that work with it. Which definitely includes FormattingLocale and FormattingOptions, but probably is a bunch of other ones, too. (This is something that it would be really nice to automate but I'm pretty sure we don't have anything for it.)


## Overview

The **Percent** control supports entering a number as a percentage while still storing the number internally as it's mathematical value. For example:
Copy link
Member

Choose a reason for hiding this comment

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

"its" rather than "it's"

- Enter 50 to represent 50% (stored as 0.5 in the DataPortal)
- Enter 72.6 to represent 72.6% (stored as 0.726 in the DataPortal)

After entering a number it is then displayed formatted using Intl.NumberFormat.
Copy link
Member

Choose a reason for hiding this comment

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

- [**Max**](../06.Setting-Details/Max.md): This defines the maximum possible numeric input into the Percent field. All input values above will display an error box to the user.
- [**DefaultValue**](../06.Setting-Details/DefaultValue.md): This defines the default value that is displayed to the user and stored in number representation on the database upon initialization.

**Note:** Min, Max, and DefaultValue should be the numeric representation of the number. For example for a max of 50%, Max should be set ot 0.5.
Copy link
Member

Choose a reason for hiding this comment

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

typo: "ot" -> "to"

"content": "Added control type for Percents to DataPortals",
"date": "1/8/2026",
"type": "major",
"changesetNum": "22614"
Copy link
Member

Choose a reason for hiding this comment

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

I think we need the product area for major updates (and I'm fine with this being a major update).

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.

3 participants