burodepeper / language-markdown

Add support for Markdown to Atom (including Github flavored, Markdown Extra, CriticMark, YAML/TOML front-matter, and R Markdown), and smart behavior to lists.
https://atom.io/packages/language-markdown
MIT License
119 stars 295 forks source link

Code blocks inside quotes not supported? #262

Open 01100001 opened 3 years ago

01100001 commented 3 years ago

This markdown text for e.g.:

Example - I:

> *NOTE:* Many (old) sources suggest adding/enabling the unstable repository
> (albeit with a low priority) to install WireGuard from, like so:
>
>     echo "deb http://deb.debian.org/debian/ unstable main" > /etc/apt/sources.list.d/unstable-wireguard.list
>     printf 'Package: *\nPin: release a=unstable\nPin-Priority: 150\n' > /etc/apt/preferences.d/limit-unstable
>
> *DON'T DO THAT* as you are pretty much opening doors for software to be
> installed and updated from unstable channel. If on Debian 11 "bullseye", which
> you should be, it's no longer even necessary!

Example - II:

> *NOTE:* Many (old) sources suggest adding/enabling the unstable repository
> (albeit with a low priority) to install WireGuard from, like so:
>
> ```
> echo "deb http://deb.debian.org/debian/ unstable main" > /etc/apt/sources.list.d/unstable-wireguard.list
> printf 'Package: *\nPin: release a=unstable\nPin-Priority: 150\n' > /etc/apt/preferences.d/limit-unstable
> ```
>
> *DON'T DO THAT* as you are pretty much opening doors for software to be
> installed and updated from unstable channel. If on Debian 11 "bullseye", which
> you should be, it's no longer even necessary!

While the emphasis (<em>) and links are identified properly, the code block is not. Is this an oversight or has it not been included due to complexity involved?

I'm kind of surprised and disappointed that Atom still doesn't have full support for Markdown grammar. For e.g. another issue that bugs me is the lack of support in grammar/parsing for emphasizing multiline text. Like:

This **won't
work** because it's complex or something for the parser to handle.

This issue bugs VS Code too, so I understand Atom isn't alone in this.

Thank you for your time!

burodepeper commented 3 years ago

I think this has been an oversight, however since this package is no longer actively maintained, I doubt it's going to be fixed.

I'm kind of surprised and disappointed that Atom still doesn't have full support for Markdown grammar. For e.g. another issue that bugs me is the lack of support in grammar/parsing for emphasizing multiline text. [...] This issue bugs VS Code too, so I understand Atom isn't alone in this.

I believe the grammars are both based off of Textmate, and it simply has some limitations. It works well enough with code, but Markdown syntax is just a little different. Multiline emphasis I believe is one of those things that just isn't doable.

01100001 commented 3 years ago

Ah, I missed the "This package is no longer actively maintained" note in the README, sorry.

You also say, "I've stopped using Atom as my primary editor, and consequently, this package." Curious about what you are using now. (Editor & package(s), if you don't mind me asking).

Thank you.