UIP icon indicating copy to clipboard operation
UIP copied to clipboard

Immediate delete wallpapers.

Open pandusonu2 opened this issue 8 years ago • 7 comments

Not sure if this is a proper issue, but I propose to add an option which deletes a downloaded wallpaper as soon as it sets it as wallpaper. Kinda like "immediate flush"

pandusonu2 avatar Nov 20 '16 19:11 pandusonu2

That could cause a serious bug on macOS, as it sets wallpaper based on file symlinking and does not copy the file. The sole purpose of this application would fail.

nkprince007 avatar Nov 20 '16 20:11 nkprince007

Then delete the previous wallpaper while applying new one? Just some way to make flush immediate rather than forced.

pandusonu2 avatar Nov 20 '16 20:11 pandusonu2

Looks like that bug I discussed about is perfectly squashed on OS X 10.11.3 and greater, so, I think we could do the immediate flush.

Or we could wait until someone from windows confirms this too. @abhsag24 @pandusonu2

nkprince007 avatar Nov 20 '16 22:11 nkprince007

windows copies the file,delete as soon as changed is better. we would have to keep downloading new wallpapers then,but i guess thats a new issue

abhay-raizada avatar Nov 21 '16 04:11 abhay-raizada

I implemented this, only to find out Ubuntu doesn't create a copy of the file.

nemani avatar Dec 10 '16 08:12 nemani

@nemaniarjun we can delete after changing, as discussed right? ohh we discussed on gitter. So we delete the wallpaper once we've changed to a different wallpaper

abhay-raizada avatar Dec 10 '16 09:12 abhay-raizada

@abhsag24 Or we could set a cronjob that eliminates old wallpapers(user sets the amount of time) if only not the current wallpaper

nkprince007 avatar Dec 10 '16 11:12 nkprince007