Skip to content

Conversation

@rfgamaral
Copy link
Member

@rfgamaral rfgamaral commented Mar 17, 2025

Overview

What the title says 👆

The goal is to make sure that the Markdown syntax used by Typist matches the most commonly accepted syntax for strike-through text, given the lack of a proper CommonMark standard. This decision also matches GitHub Flavored Markdown syntax, which is widely accepted.

PR Checklist

Test plan

  • Open the preview Storybook deployed to Netlify
  • Open the Rich-text → Default story
  • Type ~strike~ into the content control
    • Observe that the text ~strike~ appears in the editor as-is (not strike-through)

…ke-through text

BREAKING CHANGE: Removed support for the single tilde syntax (e.g., `~text~`) for strike-through formatting. Please update your content to use the double tilde syntax (e.g., `~~text~~`) instead.
@rfgamaral rfgamaral added the 👀 Show PR Used for PRs that need a review, but can be merged when CI is green. label Mar 17, 2025
@rfgamaral rfgamaral requested a review from a team March 17, 2025 12:42
@rfgamaral rfgamaral self-assigned this Mar 17, 2025
@rfgamaral rfgamaral requested review from pedroalves0 and removed request for a team March 17, 2025 12:42
@netlify
Copy link

netlify bot commented Mar 17, 2025

Deploy Preview for doist-typist ready!

Name Link
🔨 Latest commit ad28538
🔍 Latest deploy log https://app.netlify.com/sites/doist-typist/deploys/67d818b784e2f000087f656a
😎 Deploy Preview https://deploy-preview-1055--doist-typist.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 site configuration.

@rfgamaral rfgamaral merged commit 95c7231 into main Mar 17, 2025
9 checks passed
@rfgamaral rfgamaral deleted the ricardo/disable-single-tilde-strike-through branch March 17, 2025 13:02
doistbot added a commit that referenced this pull request Mar 17, 2025
## [8.0.0](v7.0.8...v8.0.0) (2025-03-17)

### ⚠ BREAKING CHANGES

* **html-serializer:** Removed support for the single tilde syntax (e.g., `~text~`) for strike-through formatting. Please update your content to use the double tilde syntax (e.g., `~~text~~`) instead.

### Bug Fixes

* **html-serializer:** Remove support for single tilde syntax for strike-through text ([#1055](#1055)) ([95c7231](95c7231))
@doistbot
Copy link
Member

🎉 This PR is included in version 8.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@doistbot doistbot added the released Pull requests that have been released to production label Mar 17, 2025
Copy link
Member

@pedroalves0 pedroalves0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Pull requests that have been released to production 👀 Show PR Used for PRs that need a review, but can be merged when CI is green.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants