Skip to content

Reformat T1, T2, T3 and suppress highlight.js #4475

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented Jun 30, 2025

Prevents unnecessary/unwanted highlight.js highlighting

Closes #4473

Note: I went for a "belt and braces" approach here with using no-highlight language-html. In theory (from reading around the subject), the first one should be sufficient, but apparently not always the case, so the latter - while technically maybe not exact - also stops any highlighting (as there's no angle brackets anywhere in the preformatted blocks)

Preview links:

@patrickhlauke
Copy link
Member Author

@kfranqueiro not sure why, but the preview build has been running for 8 minutes at this point, with no sign of concluding. might just be a temporary glitch, or something more fundamentally borked in the pipeline (I'll keep an eye on this and see if it eventually completes)

@patrickhlauke patrickhlauke force-pushed the patrickhlauke-issue4473 branch 2 times, most recently from af91c48 to 8c550fc Compare June 30, 2025 09:35
Copy link

netlify bot commented Jun 30, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit af91c48
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/68625a3aa6a8ba000894e836
😎 Deploy Preview https://deploy-preview-4475--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jun 30, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 1f44543
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/686273db07d2a400088e69ec
😎 Deploy Preview https://deploy-preview-4475--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@patrickhlauke patrickhlauke force-pushed the patrickhlauke-issue4473 branch from 8c550fc to 833fbde Compare June 30, 2025 09:40
@patrickhlauke
Copy link
Member Author

@kfranqueiro as you were...after a few force pushes, it seems to have cleared whatever blockage there was in the preview build

@patrickhlauke patrickhlauke force-pushed the patrickhlauke-issue4473 branch from 833fbde to 41187a9 Compare June 30, 2025 11:23
@patrickhlauke patrickhlauke changed the title Reformat T1 and add plaintext class Reformat T1, T2, T3 and supress highglight.js Jun 30, 2025
@patrickhlauke patrickhlauke changed the title Reformat T1, T2, T3 and supress highglight.js Reformat T1, T2, T3 and supress highlight.js Jun 30, 2025
@patrickhlauke patrickhlauke force-pushed the patrickhlauke-issue4473 branch from 41187a9 to 1f44543 Compare June 30, 2025 11:24
@patrickhlauke patrickhlauke changed the title Reformat T1, T2, T3 and supress highlight.js Reformat T1, T2, T3 and suppress highlight.js Jun 30, 2025
Copy link

netlify bot commented Jul 1, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 54fcaac
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/68623bddd7e959000859ea4d
😎 Deploy Preview https://deploy-preview-4475--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@patrickhlauke patrickhlauke requested a review from fstrr July 1, 2025 08:24
@patrickhlauke patrickhlauke self-assigned this Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

T1: Incorrect text highlighting
2 participants