Closed raucao closed 7 years ago
I think this is done for the basic functionality. Would appreciate a quick review.
When removing a bookmark with tags that are only used for that bookmark, it might be a good idea to delete those tags from the cache.
They'll be overwritten entirely next time you load the index, hence the todo method also for adding tags.
Also, when adding tags via autocompletion the last trailing comma results in an empty tag being added.
The auto-completion is only putting things in the input field, so apparently there's a bug with trailing commas in general.
I moved the tag list deletion to the service and fixed the bug with trailing slashes. Thanks for your review!
closes #29