WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@spenserblack
Copy link
Owner

@spenserblack spenserblack commented Nov 10, 2024

to do

  • Fix *foo **bar*** (probably broken by a3fb3c9)
  • Fix ***foo** bar*

These 4 combos should be tested:

text expected
***foo* bar** [b][i]foo[/i] bar[/b]
***foo** bar* [i][b]foo[/b] bar[/i]
**foo *bar*** [b]foo [i]bar[/i][/b]
*foo **bar*** [i]foo [b]bar[/b][/i]

When parsing bold text, it will search for an opening italicization
(`*`). If it's found, it then tries to find an ending italicization. If
that is found, then that text is skipped when searching for the closing
`**` for the bold text.
@spenserblack spenserblack linked an issue Nov 10, 2024 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Nov 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
steamdown ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2024 2:35am

@spenserblack spenserblack force-pushed the bugfix/233/nested-asterisks branch from 680f6a9 to 539fc3e Compare November 12, 2024 02:35
@spenserblack spenserblack marked this pull request as ready for review November 12, 2024 02:35
@spenserblack spenserblack merged commit 5d45268 into main Nov 12, 2024
6 checks passed
@spenserblack spenserblack deleted the bugfix/233/nested-asterisks branch November 12, 2024 02:37
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.

Nesting bold in italic (or vice-versa) is broken

2 participants