Skip to content

Fix disable auto closing tab when typing arrow of func #215

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

Closed
wants to merge 2 commits into from
Closed

Fix disable auto closing tab when typing arrow of func #215

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 21, 2020

Auto closing tab of > is triggered by below.

  • <Button> <- this is ok.
  • <Button on:click={() => <- this shouldn't be triggered.

I add codes to avoid second one.

ryuta69 added 2 commits June 21, 2020 12:53
Auto closing tab of '>' is triggered by below.

- `<Button>` <- this is ok.
- `<Button on:click={() =>` <- this shouldn't be triggered.

I add codes to avoid second one.
@dummdidumm
Copy link
Member

Thanks for your contribution!

We already had a PR for this which fixed the issue, so I cannot reproduce this anymore with the latest Svelte Beta. Could you tell me how to reproduce this?

@ghost
Copy link
Author

ghost commented Jun 21, 2020

[reproduce version]
jamesbirtles.svelte-vscode v0.9.3

https://marketplace.visualstudio.com/items?itemName=JamesBirtles.svelte-vscode

Oh, I thought this plugin moved to this repository, but maybe wrong? It looks only the plugin Svelte Beta is maintained.

@dummdidumm
Copy link
Member

This is the old version, Svelte Beta is the new, official and maintained one this repo is about.

@ghost
Copy link
Author

ghost commented Jun 21, 2020

Oh I am totally sorry, didn't check about that.

Thank you, and I close this.

@ghost ghost closed this Jun 21, 2020
@dummdidumm
Copy link
Member

No problem!

This pull request was closed.
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.

1 participant