Save to Disk + Bug fixes#18
Merged
Merged
Conversation
- Configurable auto-save behavior (defaults to on) for all capture modes - Fixed keybindings closing Settings when using Escape - Fixed keybindings not showing <Not Bound> after a restart - Fixed keybindings resetting to the default if cleared after a restart - Added "DELETE to reset to default" option during keybinding - Fixed an ImGui Push/Pop bug or two
- SKIV now defaults to using the same sort order as File Explorer. - Added Delete functionality to the app - Rearranged some of the context menu items -- Removed icons for some seldom used actions to improve UX -- Moved Encoder Setting further down -- Separated Copy/Close/Delete actions from the top Open/Save actions Known bugs: - File Explorer sort is retained for network shares if deleting a file - Minor focus loss bug occurs if the "Confirm Delete" warning is shown
Member
Author
|
Correction:
|
- Long-term the refreshes have to be moved to a worker thread
Member
Author
|
SKIV already removes temporary files older than a day on launch, however some users might never restart their systems so the latest commit also prunes all but the 10 newest files from the temporary folder every time a non-persistent capture is made. |
- Ctrl+S to toggle between saving to disk - Ctrl+E to toggle between selecting file in File Explorer - Fixed a bug that bound Encoder Setup to Ctrl+E instead of Ctrl+B
- Special K profile name - Product name - Executable name - Window title (not used by default)
but it seems to work, so let's find out where it takes us!
This is what is used in SKIF today and pretty much all of these changes were carried straight over from SKIF to here. This allows us to control when SKIV stores window pos/size, and specifically prevents storing unwanted values (fullscreen/snipping)
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
1 participant
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.
Uh oh!
There was an error while loading. Please reload this page.