Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: better tag handling #991

Merged
merged 3 commits into from
Oct 11, 2024
Merged

fix: better tag handling #991

merged 3 commits into from
Oct 11, 2024

Conversation

kellyjosephprice
Copy link
Collaborator

@kellyjosephprice kellyjosephprice commented Oct 10, 2024

PR App Fix RM-XYZ

🧰 Changes

Don't attempt to migrate inline html tags.

  1. remark only parses the tags, so something like:
<span>OH no!</span>

becomes:

<span />OH no!<span />
  1. This will prevent migrating custom inline tags as a side effect.

🧬 QA & Testing

This shouldn't effect production code directly, only the migration. I think it's best to test directly in the migration?

@kellyjosephprice kellyjosephprice marked this pull request as ready for review October 10, 2024 21:28
@rafegoldberg
Copy link
Contributor

rafegoldberg commented Oct 11, 2024

Also is there a reason not to do this in our new rdmdCompatParser in the monorepo?

@kellyjosephprice
Copy link
Collaborator Author

Also is there a reason not to do this in our new rdmdCompatParser in the monorepo?

Yea, should I put up a PR to accept additional compilers instead? And move this into the monorepo?

@rafegoldberg
Copy link
Contributor

@kellyjosephprice—yeah, ^that could be a nice convenience!

@kellyjosephprice kellyjosephprice merged commit e6aa82d into next Oct 11, 2024
13 checks passed
@kellyjosephprice kellyjosephprice deleted the fix/migrating-inline-html branch October 11, 2024 18:54
rafegoldberg pushed a commit that referenced this pull request Oct 14, 2024
## Version 7.6.7

### ✨ New & Improved

* **Image:** match new alignment selectors ([#992](#992)) ([73bca2b](73bca2b))

### 🛠 Fixes & Updates

* better tag handling ([#991](#991)) ([e6aa82d](e6aa82d))
* respect the top-level `copyButtons` option ([#990](#990)) ([1242413](1242413)), closes [#964](#964) [/github.com/readmeio/markdown/blob/96f9644f04e6d8e3ffff6f9c014432f901c0b804/lib/compile.ts#L28](https://github.com/readmeio//github.com/readmeio/markdown/blob/96f9644f04e6d8e3ffff6f9c014432f901c0b804/lib/compile.ts/issues/L28) [/github.com/readmeio/markdown/blob/96f9644f04e6d8e3ffff6f9c014432f901c0b804/lib/compile.ts#L28](https://github.com/readmeio//github.com/readmeio/markdown/blob/96f9644f04e6d8e3ffff6f9c014432f901c0b804/lib/compile.ts/issues/L28) [/github.com/readmeio/markdown/blob/b9502adb306f099cd91e005df17c0be252019814/options.js#L1-L22](https://github.com/readmeio//github.com/readmeio/markdown/blob/b9502adb306f099cd91e005df17c0be252019814/options.js/issues/L1-L22)

### 📘 Tests & Docs

* readme updates ([#993](#993)) ([3a4e30d](3a4e30d))

<!--SKIP CI-->
@rafegoldberg
Copy link
Contributor

This PR was released!

🚀 Changes included in v7.6.7

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants