-
Notifications
You must be signed in to change notification settings - Fork 15
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
version for Firefox #11
Comments
@lpslp This repo didn't update for many years, so I have rewritten it in React, check this https://github.com/fffx/quick-bookmark. I didn't release a firefox version for now, because I haven't decided on default key binding, as cmd+B is a firefox shortcut. The chrome version is here https://chrome.google.com/webstore/detail/quick-bookmark/bbjekmkfbdemdbfkckbakmmiceppjkdc/related |
@fffx it's nice to see that you're building on top of my work, but please note that this extension was released under MIT license, according to which you're required to keep my credentials in authors fields (in, LICENSE.md, package.json, and manifest.json) |
thanks for your work, my project structure is based on web-extension-starter, and then I rewrite it by looking at your project。so the liscense is also copy from web-extension-starter。I'm sorry for not puting your notice in the lisence file. I will do it in a few days. But why I should put your liscense notice in package.json, manifest.json, since your project even don't have a package.json file? And Where do I put mutiple author in package.json or manifest.json? |
@fffx thanks for understanding. I think putting my name back into the LICENSE.md file and creating a separate file called AUTHORS.md listing us both (in separate lines) should be enough. :) |
hi - is there a version of you great addon for Firefox?
The text was updated successfully, but these errors were encountered: