Skip to content
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

"Snippets successfully fetched" notification is annoying #25

Closed
covertbert opened this issue Jun 19, 2020 · 6 comments
Closed

"Snippets successfully fetched" notification is annoying #25

covertbert opened this issue Jun 19, 2020 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@covertbert
Copy link
Contributor

"Snippets successfully fetched" notification is kinda annoying. I find any notifications in VSCode annoying unless they're errors though so maybe that's just me...

@covertbert covertbert changed the title "Latest version" notification is annoying "Snippets successfully fetched" notification is annoying Jun 19, 2020
@lnikell
Copy link
Member

lnikell commented Jun 19, 2020

Thank you for the feedback @covertbert! We will consider removing notification on opening fetch, however, I guess having this for manual fetch via command palette still makes sense and provide good UX.

What is your overall experience with the extension?

@lnikell lnikell added the enhancement New feature or request label Jun 19, 2020
@covertbert
Copy link
Contributor Author

I agree that having feedback for the manual fetch is a good idea!

I guess my feeling about the automatic one is that it appears quite often and so it becomes noise that the brain will eventually just ignore.

Maybe it makes more sense to notify if it fails instead as this is what should matter to the user?

Just ideas anyway, maybe I’m wrong 😊

I love the extension anyway! It just needs TS support for me to really use it but I’ve added my comment about this to the other issue.

@YoussefZidan
Copy link

"Snippets successfully fetched" notification is kinda annoying. I find any notifications in VSCode annoying unless they're errors though so maybe that's just me...

I find it useful actually

@lnikell
Copy link
Member

lnikell commented Jun 22, 2020

Today we've discussed that we can add the ability to turn off notification inside Extension configuration, so it will be possible to disable them easily via UI if you find them annoying. If someone would like to be a volunteer to handle this, please let us know.

@lnikell
Copy link
Member

lnikell commented Jun 23, 2020

@covertbert Hey, we added the ability to disable notifications, you can read more about it here

@lnikell lnikell closed this as completed Jun 23, 2020
@covertbert
Copy link
Contributor Author

Thanks guys, much appreciated!

lnikell pushed a commit that referenced this issue May 25, 2021
* fix: add some fixes

* feat: add Marketplace UI without backend integration

* feat: add content for the homepage and rebuild login page

* feat: add a new Button component and start replacing the old Button with it across the whole app
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

4 participants