Skip to content

Correctly highlight links without suffix #380

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

Merged
merged 1 commit into from
Jul 23, 2022

Conversation

jez
Copy link
Contributor

@jez jez commented Jul 11, 2022

This is a revert of ad853fe. The original changes in ad853fe claimed
to be in service of this snippet:

Here is some mono space linked text: [`project`](https://github.com/vim-pandoc/vim-pandoc-syntax).
If I put here more `mono spaced` text, afterwards, the rest of the document is highlited wrong, try it yourself.

This snippet highlights completely fine, even without the changes made
in ad853fe.

On the other hand, this change prevents this snippet from highlighting
the URL in the [anchor] at the bottom:

This is an [anchor] tag that should be auto-linked.

[anchor]: https://example.com

I would love to revert the original change. Reverting it seems purely
additive to me (does not break old behaviors, adds new behaviors).

Before

Screen Shot 2022-07-11 at 10 51 13 AM

After

Screen Shot 2022-07-11 at 10 51 28 AM

Before

Screen Shot 2022-07-11 at 10 56 46 AM

After

Screen Shot 2022-07-11 at 10 56 32 AM

Fixes #289

This is a revert of ad853fe. The original changes in ad853fe claimed
to be in service of this snippet:

    Here is some mono space linked text: [`project`](https://github.com/vim-pandoc/vim-pandoc-syntax).
    If I put here more `mono spaced` text, afterwards, the rest of the document is highlited wrong, try it yourself.

This snippet highlights completely fine, even without the changes made
in ad853fe.

On the other hand, this change prevents this snippet from highlighting
the `[anchor]` inside the sentence:

    This is an [anchor] tag that should be auto-linked.

    [anchor]: https://example.com

I would love to revert the original change. Reverting it seems purely
additive to me (does not break old behaviors, adds new behaviors).
@fmoralesc
Copy link
Member

LGTM too. Thanks!

@fmoralesc fmoralesc merged commit 2baeabb into vim-pandoc:master Jul 23, 2022
@jez jez deleted the jez-revert-anchor branch July 24, 2022 01:54
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.

Regression in pandocReferenceDefinition
2 participants