Merge pull request #2072 from omnivore-app/fix/labels-on-reader

Fix labels not being displayed on the reader
This commit is contained in:
Jackson Harper
2023-04-18 15:00:55 +08:00
committed by GitHub

View File

@ -118,16 +118,21 @@ export function ArticleContainer(props: ArticleContainerProps): JSX.Element {
props.highlightOnRelease
)
// iOS app embed can overide the original margin and line height
const [maxWidthPercentageOverride, setMaxWidthPercentageOverride] =
useState<number | null>(null)
const [lineHeightOverride, setLineHeightOverride] =
useState<number | null>(null)
const [fontFamilyOverride, setFontFamilyOverride] =
useState<string | null>(null)
const [highContrastTextOverride, setHighContrastTextOverride] =
useState<boolean | undefined>(undefined)
const [justifyTextOverride, setJustifyTextOverride] =
useState<boolean | undefined>(undefined)
const [maxWidthPercentageOverride, setMaxWidthPercentageOverride] = useState<
number | null
>(null)
const [lineHeightOverride, setLineHeightOverride] = useState<number | null>(
null
)
const [fontFamilyOverride, setFontFamilyOverride] = useState<string | null>(
null
)
const [highContrastTextOverride, setHighContrastTextOverride] = useState<
boolean | undefined
>(undefined)
const [justifyTextOverride, setJustifyTextOverride] = useState<
boolean | undefined
>(undefined)
const highlightHref = useRef(
window.location.hash ? window.location.hash.split('#')[1] : null
)
@ -142,8 +147,9 @@ export function ArticleContainer(props: ArticleContainerProps): JSX.Element {
)
useEffect(() => {
setLabels(props.labels)
updateFontSize(props.fontSize ?? 20)
}, [props.fontSize])
}, [props.labels, props.fontSize])
// Listen for preference change events sent from host apps (ios, macos...)
useEffect(() => {