Skip to content
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

Implement support for rendering /italics/ properly #25

Open
4 tasks
novoid opened this issue Sep 30, 2018 · 2 comments
Open
4 tasks

Implement support for rendering /italics/ properly #25

novoid opened this issue Sep 30, 2018 · 2 comments
Labels
enhancement helpwanted quickfix small change necessary to fix this

Comments

@novoid
Copy link
Owner

novoid commented Sep 30, 2018

This seems to be a tough one: implement support for italics within blog posts without breaking support for determining URLs.

To be honest, I did not spend much time investigating how Org mode is implemented. Maybe there are some clever ideas how to do it properly. So far, I did not do it. Maybe it's easily done.

Ideas for unit tests:

  • determining italics within a line that holds URLs as well
  • determining italics that span over one or even more line breaks (Org mode does it only for one line break AFAIR)

And:

@xeruf
Copy link
Contributor

xeruf commented Sep 8, 2022

  • I don't think italics in org mode can span over linebreaks
  • URLs don't start with a slash, isn't that a clear indicator?

@novoid novoid added the quickfix small change necessary to fix this label Sep 8, 2022
@novoid
Copy link
Owner Author

novoid commented Sep 8, 2022

Emacs Org-mode does support italics where a maximum of one line break is in-between. I just tested it via emacs -q

I might not be able to fix this with the current parser. Maybe #63 would be the next chance?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement helpwanted quickfix small change necessary to fix this
Projects
None yet
Development

No branches or pull requests

2 participants