Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paragraph Font Family overridden by Global Style Link Font Family for links in the Paragraph #74280

Closed
ccwalburn opened this issue Mar 9, 2023 · 3 comments
Labels
[Feature] Global Styles The Global Styles tools in the site editor and theme style variations. [Platform] Atomic [Platform] Simple [Pri] Normal Schedule for the next available opportuinity. [Status] Core Fix Needed A fix within the Core WordPress or Gutenberg project is required to resolve this issue. Triaged To be used when issues have been triaged. [Type] Bug User Report This issue was created following a WordPress customer report

Comments

@ccwalburn
Copy link

Quick summary

When a Paragraph has a link in it, the link does not use the Font Family set for the Paragraph. It uses the link font family as set in Global Styles.

Steps to reproduce

I used the Zaino theme

  1. Go to Appearance > Editor and edit the Home template
  2. Add a Paragraph block
  3. Highlight a few words in the Paragraph block and make them a link
  4. Change the Font Family for the Paragraph block to Texturina
  5. Go to Styles > Typography > Links
  6. Change the font to EB Garamond
  7. Save the template

What you expected to happen

I expected the entire paragraph, including links, to use the Texturina font

What actually happened

The links used the EB Garamond font

Impact

All

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

No response

Logs or notes

I added this CSS as a workaround:

.has-texturina-font-family a:where(:not(.wp-element-button)) {
    font-family: var(--wp--preset--font-family--texturina) !important;
}

6028752-zen

@ccwalburn ccwalburn added [Type] Bug User Report This issue was created following a WordPress customer report labels Mar 9, 2023
@github-actions github-actions bot added the [Pri] Normal Schedule for the next available opportuinity. label Mar 9, 2023
@github-actions
Copy link

github-actions bot commented Mar 9, 2023

Support References

This comment is automatically generated. Please do not edit it.

  • 6028752-zen

@cuemarie
Copy link

📌 SCRUBBING : RESULT - Replicated / Could Not Replicate / Uncertain

  • Tested on Simple: Replicated
  • Tested on AT: Replicated
  • Tested on Self-hosted:
  • Replicable on Core? Yes

📌 FINDINGS/SCREENSHOTS/VIDEO

  • Replicated in all 3 testing environments with FSE themes.

Simple Example

6ThHmC.mp4

📌 ACTIONS

  • Marked as Triaged for Quality Squad review

📌 NOTES

@cuemarie cuemarie added [Status] Core Fix Needed A fix within the Core WordPress or Gutenberg project is required to resolve this issue. Triaged To be used when issues have been triaged. [Platform] Simple [Platform] Atomic [Feature] Global Styles The Global Styles tools in the site editor and theme style variations. labels Mar 12, 2023
@dsas
Copy link
Contributor

dsas commented Mar 27, 2023

@cuemarie, that's right. This issue might explain a bit more directly: WordPress/gutenberg#44931

Basically GB is working as intended, but has a confusing UX. I'm going to close this as not specifically related to Dotcom.

Thank you for reporting @ccwalburn 🙇‍♂️

@dsas dsas closed this as not planned Won't fix, can't repro, duplicate, stale Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Global Styles The Global Styles tools in the site editor and theme style variations. [Platform] Atomic [Platform] Simple [Pri] Normal Schedule for the next available opportuinity. [Status] Core Fix Needed A fix within the Core WordPress or Gutenberg project is required to resolve this issue. Triaged To be used when issues have been triaged. [Type] Bug User Report This issue was created following a WordPress customer report
Projects
None yet
Development

No branches or pull requests

3 participants