-
Notifications
You must be signed in to change notification settings - Fork 27
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
Update to manifest v3, redesign UI to allow click-to-archive + tagging, add history view + options.html page for settings #31
base: master
Are you sure you want to change the base?
Conversation
…marks and history
cc @tjhorner FYI, let me know if you have any questions/comments/concerns, would love to get your input |
Also fix closing popup when pressing escape.
Hey @benmuth can you open a PR against this |
Also cleaned up now unused code from popup.js.
Co-authored-by: Nick Sweeting <github@sweeting.me>
Also some style fixes.
@benmuth can you confirm if this is working against both |
The logic used to require that an API key was set in the extension configuration even if version 0.7.3 was being used.
@benmuth Can you help me do some final bugfixes on this PR. Then I think it's ready to release soon!
(Also plz send an invoice for your time) This extension is going to be reworked at some point to make it work as a desktop app too (I'm testing out Quasar+vite as a new frontend framework). |
@pirate Should the 'Sync' button work with older versions of ArchiveBox as well? I can repeat the logic we used in the other redesigned code. |
yes, use the same logic / call the same function |
With help from @benmuth and Claude.
Usage:
It should autoreload from source automatically for most basic edits, but it can also be manually refreshed from
chrome://extensions
.TODO Add more configuration options and allow setting per-URL config via tags likewill revesit laterdepth=1
#4TODO Indicator that the current webpage is archived #29will revisit laterTODO List archives in a side bar #15will revisit later/api/v1/cli/add
, use/add
instead?