issue-18714: Replaced deprecated constants with enum in EthSignModal and HoldToRevealModal Component #19499
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Explanation
Issue link: #18714
Replaced deprecated constants with enum in
eth-sign-modal.js
andhold-to-reveal-modal.js
Constants replaced with enum:
DISPLAY
->Display
FLEX_DIRECTION
->FlexDirection
SEVERITIES
->Severity
Screenshots/Screencaps
Before
eth-sign-modal.js
:hold-to-reveal-modal.js
:After
eth-sign-modal.js
:hold-to-reveal-modal.js
:Manual Testing Steps
yarn storybook
localhost:6006
in browserEthSignModal
andHoldToRevealModal
componentPre-merge author checklist
Pre-merge reviewer checklist
If further QA is required (e.g. new feature, complex testing steps, large refactor), add the
Extension QA Board
label.In this case, a QA Engineer approval will be be required.