Skip to content

bug: diff code blocks used but not supported (#4156) #10

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

Merged
merged 3 commits into from
Jun 18, 2025

Conversation

soundproofboot
Copy link

resolves ionic-team#4156

  • Add 'diff' to list of additional languages supported in docusaurus.config.js.
  • Add additional styles in styles/components/_code.scss for inserted and deleted lines in diff code blocks only.
diffHighlightingFix

@perrygovier
Copy link
Owner

Let's go with #f55073 for the red and #42b983 for the green. after that, feel free to send this on to the ionic team as well.

@soundproofboot
Copy link
Author

Question on the css selectors - are .token.inserted and .token.deleted selectors in src/styles/components/_code.scss used by any other elements or are they there only for the diff blocks and just previously unused?

@perrygovier
Copy link
Owner

Screenshot 2025-06-18 at 2 32 22 PM Looks good. Here's a list of pages found to have `class="` then the word `token` followed by the word `inserted` then followed by a closing `"`. The screenshot also includes the regex I used. Looking at these pages, these have code blocks we'd expect to have these selectors.

@perrygovier perrygovier merged commit 1082761 into perrygovier:main Jun 18, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: diff code blocks used but not supported
2 participants