-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Indication that file compression was reverted #245
Comments
I thought that maybe we could add the indication on each row? Instead of the percentage of reduction. |
The indicator on each row is a better idea. Removing the skipping params would make implementing this easier, so maybe that's what we should do. What should the indicator be? Text that just says "Skipped" or "Reverted"? |
Or maybe instead of removing the skipping params, we could just check the old size against the new size and if new size is equal or superior we could write this text. "Skipped" is perfect for me. |
Checking the size would probably be better. I also think "Skipped" is a good indicator, but this is technically incorrect for the system when Curtail is in overwrite mode. Since it overwrites the file first, that's technically reverting the file. Maybe the system could be updated to compress the temp file first, check if that's bigger or smaller, and then copy that file onto the original if it's smaller. This would allow for skipping the file in overwrite mode, but it would also mean successful compression would have one additional step. Would that be okay with you? |
It seems like a good idea for me! If you want to work on it and if you want some help let me know :) |
I'll work on it when I get the chance, and I'll ask you if I need help. Thanks :) |
If compression was skipped for a file, the file's row will communicate that it was. There will also be a more information button that will explain why the file was skipped. Closes Huluti#245
If compression was skipped for a file, the file's row will communicate that it was. There will also be a more information button that will explain why the file was skipped. Closes Huluti#245
If compression was skipped for a file, the file's row will communicate that it was. There will also be a more information button that will explain why the file was skipped. Closes Huluti#245
Follow-up to #229. Maybe this could be a toast? Another thing to consider is, should this happen only when Curtail manually reverts the file, or also when the compressors skip the compression? The latter might be harder to figure out.
The text was updated successfully, but these errors were encountered: