Closed Nick-1979 closed 2 days ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request involve the addition of a new prop, withSmallDecimal
, to the FormatPrice
component across multiple files. This prop enables conditional rendering of smaller decimal values in various components that utilize FormatPrice
, such as TotalChart
, AccountInformationForHome
, TotalBalancePieChart
, and YouHave
. Additionally, a new utility function, getDecimal
, has been introduced for extracting decimal places, and existing functions have been modified to enhance decimal handling and error logging.
File Path | Change Summary |
---|---|
packages/extension-polkagate/src/components/FormatPrice.tsx |
Added withSmallDecimal prop to Props interface and updated function signature. Adjusted decimal point logic based on withSmallDecimal . Introduced reduceFontSize function. |
packages/extension-polkagate/src/fullscreen/accountDetails/components/TotalChart.tsx |
Updated FormatPrice component call to include withSmallDecimal prop for displaying total worth. |
packages/extension-polkagate/src/fullscreen/homeFullScreen/partials/AccountInformationForHome.tsx |
Updated FormatPrice component in AccountTotal to include withSmallDecimal prop for total balance display. |
packages/extension-polkagate/src/fullscreen/homeFullScreen/partials/TotalBalancePieChart.tsx |
Added withSmallDecimal prop to FormatPrice component for portfolio value formatting. Adjusted CountUp component to dynamically set decimals prop. |
packages/extension-polkagate/src/popup/home/YouHave.tsx |
Updated FormatPrice component to include withSmallDecimal prop for price rendering. |
packages/extension-polkagate/src/util/utils.ts |
Introduced getDecimal function for extracting decimal places. Modified fixFloatingPoint and enhanced error handling in getSubstrateAddress and updateRecentChains . |
withSmallDecimal
to the FormatPrice
component, which is also referenced in the retrieved PRs, indicating a direct relationship in the changes made to the same component.FormatPrice
component, specifically regarding the display of price changes, which may relate to the new withSmallDecimal
functionality.FormatPrice
component and its new props, including withSmallDecimal
.CountUp
feature in the FormatPrice
component may relate to the changes in how prices are displayed, including the handling of small decimals.TotalBalancePieChart
component also reference the FormatPrice
component, indicating a connection to the changes made in the main PR.🐰 In the meadow, prices dance,
With small decimals, they take a chance.
FormatPrice shines, oh what a sight,
With new props added, all feels right!
Hop along, let numbers play,
In our code, they’ll frolic all day! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
New Features
withSmallDecimal
prop in theFormatPrice
component, allowing for conditional rendering of smaller decimal values across several components.Bug Fixes
Documentation
withSmallDecimal
prop.