Do not use |- ligature when part of |-|

[andOlga]

The |-| sequence of characters is used in Markdown tables to separate the header of the table from the contents, e.g.:

|Column 1|Column 2|
|-|-|
|R1C1|R1C2|
|R2C1|R2C2|

Currently, Fira code applies the |- ligature here, resulting in this, which looks entirely wrong:

[tonsky]

thanks, will adjust!

[AnandChowdhary]

@tonsky, this could be an interesting exploration of Fira Code ligatures for markdown. Also curious about your thoughts, @ooa113y.

Current behavior:

With proposal ligatures:

I understand that multi-line ligatures might be impossible, but maybe something like this?

This can be ligatures for | - and - |, for example. Also, - | - could render to something similar to a wider +.

If this is something you'd consider, I can open another issues to discuss it further.

[wopian]

@AnandChowdhary: I understand that multi-line ligatures might be impossible, but maybe something like this?

I don't think this is a good idea for a ligature.

I would have no idea if I (or the previous committer) typed | -- | -- | or |----|----|, which could lead to people committing something like | -- |--- | ---| unintentionally.

And for myself, it would make resolving a linter error annoying where a space between table columns is required in my projects. |---| would throw an error, which wouldn't be obvious from looking as it would be visually identical to | - |.

[tonsky]

Current Fira Code should render like this: