Skip to content

Minor update README.md #9

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
Feb 16, 2016
Merged

Conversation

1zaak
Copy link
Contributor

@1zaak 1zaak commented Feb 12, 2016

Thought would be helpful to mention the open tag because most snippets doesn't require it to be typed.

@stephnr
Copy link
Contributor

stephnr commented Feb 12, 2016

This is incorrect. The snippets are not designed/triggered by a leading caret. The HTML parsers written in Sublime natively parse out the carets when reading snippets

Is this proposed because of something you experienced?

@izaakhaliq

@1zaak
Copy link
Contributor Author

1zaak commented Feb 12, 2016

Yup it was definitely what I experienced. Not sure what went wrong with my packages there. Thanks anyway for the info @Stephn-R

@christiaanwesterbeek
Copy link
Member

In an html file, when I type md- nothing happens. When I type <md- I get the snippets menu showing md- at the top, which is fine. Now, when I click it, I end up with this snippet. And notice the double caret in the beginning.

<<md-tab label="" disabled md-on-select="" md-on-deselect="">...</md-tab>

@stephnr
Copy link
Contributor

stephnr commented Feb 13, 2016

Ah so this never happened before. Perhaps there was a bug introduced with
the latest release of Sublime
On Sat, Feb 13, 2016 at 3:15 PM Christiaan Westerbeek <
notifications@github.com> wrote:

In an html file, when I type md- nothing happens. When I type <md- I get
the snippets menu showing md- at the top, which is fine. Now, when I click
it, I end up with this snippet. And notice the double caret in the
beginning.

<...


Reply to this email directly or view it on GitHub
#9 (comment)
.

@stephnr
Copy link
Contributor

stephnr commented Feb 16, 2016

Ok, confirmed. I will be pushing up a change shortly

@christiaanwesterbeek
Copy link
Member

Actually it happened with me since I started this snippet library. I'm gonna merge both pull requests and push a new version somewhere this week.

christiaanwesterbeek added a commit that referenced this pull request Feb 16, 2016
@christiaanwesterbeek christiaanwesterbeek merged commit 24c1491 into devotis:master Feb 16, 2016
@stephnr
Copy link
Contributor

stephnr commented Feb 16, 2016

Thanks @devotis 🍻

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.

3 participants