Curtail icon indicating copy to clipboard operation
Curtail copied to clipboard

Indication that file compression was reverted

Open ARAKHN1D opened this issue 1 year ago • 6 comments

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.

ARAKHN1D avatar Sep 22 '24 14:09 ARAKHN1D

I thought that maybe we could add the indication on each row? Instead of the percentage of reduction. Maybe we could remove all the params that skip image if larger than original and rely on the system you added for all compressors?

Huluti avatar Sep 27 '24 19:09 Huluti

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"?

ARAKHN1D avatar Sep 28 '24 00:09 ARAKHN1D

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. What do you think?

Huluti avatar Sep 28 '24 07:09 Huluti

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?

ARAKHN1D avatar Sep 28 '24 16:09 ARAKHN1D

It seems like a good idea for me! If you want to work on it and if you want some help let me know :)

Huluti avatar Oct 02 '24 11:10 Huluti

I'll work on it when I get the chance, and I'll ask you if I need help. Thanks :)

ARAKHN1D avatar Oct 02 '24 20:10 ARAKHN1D