Skip to content

Conversation

@urda
Copy link
Contributor

@urda urda commented Sep 24, 2017

I love the "copy to clipboard" power of Boostnote within code blocks. However, as an end user, I don't really care for Boostnote to display a notification on copy.

This PR adds a configuration option to disable these notifications! The new option is show below

screen shot 2017-09-23 at 5 03 47 pm

This is my first PR to Boostnote, please let me know if this commit needs amended.

@urda
Copy link
Contributor Author

urda commented Sep 24, 2017

Fixed the travis build, let me know if you have any questions. Hopefully this gets merged in to the next release, I would love to start using this feature.

@kazup01
Copy link
Member

kazup01 commented Sep 24, 2017

Thanks @urda ! We will check it.

@urda
Copy link
Contributor Author

urda commented Sep 24, 2017

Great! Thanks for an awesome note taking app!

@kazup01
Copy link
Member

kazup01 commented Sep 24, 2017

Thanks for your cooperation, too :)

@urda
Copy link
Contributor Author

urda commented Oct 1, 2017

Re-sync'ing with upstream/master after a week of commits, still waiting for a review / feedback on my PR 😃

Copy link
Contributor

@asmsuechan asmsuechan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine on macOS and ubuntu. Thanks!

@asmsuechan asmsuechan merged commit 95d74d1 into BoostIO:master Oct 1, 2017
@urda
Copy link
Contributor Author

urda commented Oct 2, 2017

Thanks! Can’t wait to see it released :)

@kohei-takata kohei-takata mentioned this pull request Oct 28, 2017
@kazup01
Copy link
Member

kazup01 commented Oct 28, 2017

This feature has just released at v0.8.16. Thanks @urda !
#1027

@kazup01
Copy link
Member

kazup01 commented Oct 29, 2017

@urda
Copy link
Contributor Author

urda commented Oct 29, 2017

Sounds good @kazup01 thanks!

@urda urda deleted the urda/feature/copy-notify-toggle branch November 20, 2017 01:59
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.

4 participants