Skip to content

Commit

Permalink
Don't reset error
Browse files Browse the repository at this point in the history
  • Loading branch information
katspaugh committed Jul 3, 2023
1 parent 740ba6d commit 27ce395
Showing 1 changed file with 10 additions and 17 deletions.
27 changes: 10 additions & 17 deletions src/components/tx/security/redefine/index.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useContext, useEffect, useRef } from 'react'
import { useContext, useEffect, useRef, useState } from 'react'
import { mapRedefineSeverity } from '@/components/tx/security/redefine/useRedefine'
import { TxSecurityContext } from '@/components/tx/security/shared/TxSecurityContext'
import { SecuritySeverity } from '@/services/security/modules/types'
Expand All @@ -22,17 +22,10 @@ import { RedefineHint } from '@/components/tx/security/redefine/RedefineHint'
const MAX_SHOWN_WARNINGS = 3

const RedefineBlock = () => {
const {
severity,
isLoading,
error,
needsRiskConfirmation,
isRiskConfirmed,
setIsRiskConfirmed,
isRiskIgnored,
setIsRiskIgnored,
} = useContext(TxSecurityContext)
const { severity, isLoading, error, needsRiskConfirmation, isRiskConfirmed, setIsRiskConfirmed, isRiskIgnored } =
useContext(TxSecurityContext)
const checkboxRef = useRef<HTMLElement>(null)
const [highlightCheckbox, setHighlightCheckbox] = useState(false)

const isDarkMode = useDarkMode()
const severityProps = severity !== undefined ? mapRedefineSeverity[severity] : undefined
Expand All @@ -43,12 +36,12 @@ const RedefineBlock = () => {

// Highlight checkbox if user tries to submit transaction without confirming risks
useEffect(() => {
if (isRiskIgnored && checkboxRef.current) {
checkboxRef.current.scrollIntoView({ behavior: 'smooth', block: 'center' })
const timeout = setTimeout(() => setIsRiskIgnored(false), 3000)
return () => clearTimeout(timeout)
setHighlightCheckbox(false)
if (isRiskIgnored) {
setTimeout(() => setHighlightCheckbox(true), 100)
checkboxRef.current?.scrollIntoView({ behavior: 'smooth', block: 'center' })
}
}, [isRiskIgnored, setIsRiskIgnored, checkboxRef])
}, [isRiskIgnored, checkboxRef])

return (
<div className={css.wrapperBox}>
Expand Down Expand Up @@ -104,7 +97,7 @@ const RedefineBlock = () => {
<FormControlLabel
label="I understand the risks and would like to continue this transaction"
control={<Checkbox checked={isRiskConfirmed} onChange={toggleConfirmation} />}
className={isRiskIgnored ? css.checkboxError : ''}
className={highlightCheckbox ? css.checkboxError : ''}
/>
</Track>
</Box>
Expand Down

0 comments on commit 27ce395

Please sign in to comment.