Add more TODO comments
This commit is contained in:
@@ -55,6 +55,9 @@ const ImageItem = ({imageSrc, onZoom, onRequestClose}: Props) => {
|
|||||||
)
|
)
|
||||||
|
|
||||||
const [translate, scale] = getImageTransform(imageDimensions, SCREEN)
|
const [translate, scale] = getImageTransform(imageDimensions, SCREEN)
|
||||||
|
|
||||||
|
// TODO: It's not valid to reinitialize Animated values during render.
|
||||||
|
// This is a bug.
|
||||||
const scrollValueY = new Animated.Value(0)
|
const scrollValueY = new Animated.Value(0)
|
||||||
const scaleValue = new Animated.Value(scale || 1)
|
const scaleValue = new Animated.Value(scale || 1)
|
||||||
const translateValue = new Animated.ValueXY(translate)
|
const translateValue = new Animated.ValueXY(translate)
|
||||||
|
|||||||
@@ -56,6 +56,8 @@ const usePanResponder = ({
|
|||||||
let isDoubleTapPerformed = false
|
let isDoubleTapPerformed = false
|
||||||
let lastTapTS: number | null = null
|
let lastTapTS: number | null = null
|
||||||
|
|
||||||
|
// TODO: It's not valid to reinitialize Animated values during render.
|
||||||
|
// This is a bug.
|
||||||
const scaleValue = new Animated.Value(initialScale)
|
const scaleValue = new Animated.Value(initialScale)
|
||||||
const translateValue = new Animated.ValueXY(initialTranslate)
|
const translateValue = new Animated.ValueXY(initialTranslate)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user