Skip to content

Comments

Add display settings UI & format utilities#163

Merged
BlobMaster41 merged 5 commits intomasterfrom
feat/conversion-clarity
Feb 21, 2026
Merged

Add display settings UI & format utilities#163
BlobMaster41 merged 5 commits intomasterfrom
feat/conversion-clarity

Conversation

@BlobMaster41
Copy link

@BlobMaster41 BlobMaster41 commented Feb 19, 2026

Description

Add new display settings UI and formatting utilities: introduces DisplaySetupPopup component and DisplaySettingsScreen, plus integrations in SettingsTabScreen and MainRoute. Add BigintToString and formatAmount utilities with tests and update OpNet-related bigint formatting and OpNetBalanceCard to use them. Update settings state (hooks and reducer) to persist display preferences. Also update package.json and tsconfig (and regenerate lockfile) for dependency/tooling bumps.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Performance improvement
  • Refactoring (no functional changes)
  • Documentation update
  • CI/CD changes
  • Dependencies update

Checklist

Build & Tests

  • npm install completes without errors
  • npm run build:chrome builds successfully
  • Extension loads without errors in browser

Code Quality

  • Code follows the project's coding standards
  • No new TypeScript/ESLint warnings introduced
  • Error handling is appropriate
  • Console logs removed (except for error logging)

Documentation

  • Code comments added for complex logic
  • README updated (if applicable)

Security

  • No sensitive data (keys, credentials) committed
  • No new security vulnerabilities introduced
  • Private keys are never logged or exposed
  • User inputs are properly validated

OPWallet Specific

  • Changes work across all supported browsers (Chrome, Firefox, Brave, Edge, Opera)
  • Wallet state management is handled correctly
  • Transaction signing follows security best practices
  • RPC communication is secure
  • Content script isolation is maintained
  • Background/popup communication is secure

Testing

Browser Testing

  • Chrome
  • Firefox
  • Brave
  • Edge
  • Opera

Screenshots

Related Issues


By submitting this PR, I confirm that my contribution is made under the terms of the project's license.

Add new display settings UI and formatting utilities: introduces DisplaySetupPopup component and DisplaySettingsScreen, plus integrations in SettingsTabScreen and MainRoute. Add BigintToString and formatAmount utilities with tests and update OpNet-related bigint formatting and OpNetBalanceCard to use them. Update settings state (hooks and reducer) to persist display preferences. Also update package.json and tsconfig (and regenerate lockfile) for dependency/tooling bumps.
@BlobMaster41 BlobMaster41 added bug Something isn't working enhancement New feature or request labels Feb 19, 2026
Add vitest.config.ts and update package.json/package-lock.json to use Vitest and related packages; remove Jest-related dependencies (and testing-library jest-dom). Move unit test files from src/... to tests/... (renamed/moved BigintToString.test.ts and formatAmount.test.ts). Also includes minor dependency adjustments in package-lock (e.g. opnet version change).
Refactor amount formatting to use BigNumber for precision and to never emit scientific notation (BigNumber.config EXPONENTIAL_AT set). formatAmount now parses inputs into BigNumber, applies K/M/B scaling, comma insertion, and uses ROUND_DOWN for fixed decimals; removed the old formatFixed helper. BigintToString dust handling updated to compute decimal places and return a trimmed toFixed string (never scientific). DisplaySetupPopup now hides the setup UI in notification windows and when the app is locked (imports useIsUnlocked and getUiType). Tests updated to expect non-scientific representation for very small/dust values.
Update package.json and regenerate package-lock.json to apply dependency upgrades. Notable bumps include @types/chrome, @types/node, glob, i18next, opnet (rc.7), postcss-preset-env, rollup (4.58.0) and multiple @csstools/postcss packages; also adds @csstools/postcss-font-width-property. This commit syncs the lockfile with the updated package manifest and applies transitive package version updates across the dependency tree.
@BlobMaster41 BlobMaster41 merged commit 44c5aee into master Feb 21, 2026
2 checks passed
@BlobMaster41 BlobMaster41 deleted the feat/conversion-clarity branch February 21, 2026 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant