Feature/copy images and tags to clipboard, "Inherited tag" and "saving" optimizations, and bug fixes.#51
Merged
Merged
Conversation
…t in focus. Switch from navigator.clipboard.write to use electron's clipboard.writeImage(). * Add toasts informing the copying status when copy image to clipboard.
…nsion for performance. Return logic that used canvas to convert the image data as a fallback, since nativeImage is not compatible with some extensions.
* Disable copy image buttons to video files.
*Add variables and methods to UiStore, File, and TagStore. *Add menu options to copy and paste tags to file contextual menus.
…imes when a large number of files is selected: from ~40,000ms to ~7,000ms while selecting ~17,000 files with tags. * Cached tag `impliedAncestors` using computed. * Switched to arrays instead of sets when possible, since arrays iterate faster. * Avoided the most unnecessary iterations, conversions between arrays and sets, or recreations.
… app is saving files to database * Prevent broken files to decrease untagged count then trying to add tags to them
…n before previous save completes * Copy and clean the filesToSave list before saving to the backend. Prevents sending files multiple times if saveFilesToSave is triggered again before the first promise resolves. * Set isSaving to false only after all file-saving promises are complete.
…mpletely the file visible inherited tags calculation and just return the file's tags.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Things added while testing the other features: