Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Don't save to existing location if not modified #94

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Don't save to existing location if not modified #94

wants to merge 1 commit into from

Conversation

jamesgpearce
Copy link

Holding down cmd-S too long will cause multiple synchronous saves which can be expensive if there is heavyweight linter activity listening for the save events.

This simply debounces it (in a sense) by not saving it the file if it is known not to have changed. Also this has no effect on saving new files for the first time.

Holding down cmd-S too long will cause multiple synchronous saves which can be expensive if there is heavyweight linter activity listening for the save events.

This simply debounces it (in a sense) by not saving it the file if it is known not to have changed. Also this has no effect on saving new files for the first time.
@jamesgpearce
Copy link
Author

bounced travis and all is good... please review!

@winstliu
Copy link
Contributor

Please add specs to this :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants