Merge pull request #1751 from Samrat-Saha-Sammy/main

fix(#1750): fixed downloaded PDF have issue(s) with Typography options
This commit is contained in:
Amruth Pillai
2024-03-10 10:13:09 +01:00
committed by GitHub
2 changed files with 6 additions and 13 deletions

View File

@ -49,17 +49,10 @@ export const ArtboardPage = () => {
// Typography Options
useEffect(() => {
if (metadata.typography.hideIcons) {
document.querySelector("#root")!.classList.add("hide-icons");
} else {
document.querySelector("#root")!.classList.remove("hide-icons");
}
if (metadata.typography.underlineLinks) {
document.querySelector("#root")!.classList.add("underline-links");
} else {
document.querySelector("#root")!.classList.remove("underline-links");
}
document.querySelectorAll(`[data-page]`).forEach((el) => {
el.classList.toggle("hide-icons", metadata.typography.hideIcons);
el.classList.toggle("underline-links", metadata.typography.underlineLinks);
});
}, [metadata]);
return <Outlet />;

View File

@ -12,11 +12,11 @@
@apply antialiased;
}
#root.hide-icons .ph {
[data-page].hide-icons .ph {
@apply hidden;
}
#root.underline-links a {
[data-page].underline-links a {
@apply underline underline-offset-2;
}