Skip to content
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

Auto delete '*-html5ified-dummy.mp4' files #541

Closed
3 tasks done
Shakil-Shahadat opened this issue Dec 11, 2020 · 18 comments
Closed
3 tasks done

Auto delete '*-html5ified-dummy.mp4' files #541

Shakil-Shahadat opened this issue Dec 11, 2020 · 18 comments

Comments

@Shakil-Shahadat
Copy link
Contributor

  • I have tested the newest version
  • I have checked readme and Known issues / Troubleshooting
  • I have searched for similar issues

The auto delete can be executed when the related file is closed or LC is closed.

Many people may want to keep that file, thus an option would be nicer.

@mifi
Copy link
Owner

mifi commented Dec 11, 2020

Currently it will be deleted when you press the Trash source button #236

@Shakil-Shahadat
Copy link
Contributor Author

Clicking on trash deletes both dummy file and source file. Don't wanna delete source file though.

@mifi
Copy link
Owner

mifi commented Dec 14, 2020

Some people don't want to auto delete dummy. I will make a dialog when pressing trash source button:

Screenshot 2020-12-14 at 16 15 37

mifi added a commit that referenced this issue Dec 14, 2020
@Shakil-Shahadat
Copy link
Contributor Author

Trash button working great so far. A couple of observations here.

  1. The options in the dialogue box are not left-aligned like the image you posted, see mine.
    Capture
  2. The default option is a bit dangerous / intrusive. If anyone becomes a bit careless will delete the source file. I think the first option could be a safer default.
  3. Button titles can be simplified and made more recognizable. I suggest renaming them to just 'Trash' & 'Cancel'.

Shakil-Shahadat added a commit to Shakil-Shahadat/lossless-cut that referenced this issue Jan 5, 2021
Button titles simplified and made more recognizable. See mifi#541 no 3.
Shakil-Shahadat added a commit to Shakil-Shahadat/lossless-cut that referenced this issue Jan 5, 2021
Changed the default option to a safer one. See mifi#541 point 2.
@markusd1984
Copy link

@Shakil-Shahadat I agree, except the wording why not call it "Delete" instead of trash?
It seems rather odd, unless it's an american thing, delete is so more commonly used (just like in the second button "do not delete").

@mattack1
Copy link

mattack1 commented Jan 5, 2021

Because at least according to the dialog, it is moving the files to the trash and NOT deleting them. i.e. the "garbage can" icon.. (I think Windows calls it recycling?)

this way, the user can manually recover them before emptying the trash.

@markusd1984
Copy link

Ah sry I mixed it up with previously deleting source file... Perhaps 'Move to recycle bin' would be better.

@mattack1
Copy link

mattack1 commented Jan 5, 2021

But it's not called recycle on a Mac, that's what I meant.. so "Trash" is kind of generic.. unless you do different UI for different versions.. it would be trash on a mac.. (e.g. "Move to Trash" in the Finder).

@Shakil-Shahadat
Copy link
Contributor Author

@markusd1984 Exactly as @mattack1 said.

Anyway, I sent two pr, if anyone can verify them, that would be nice. My build is not working on my machine.

@markusd1984
Copy link

@mattack1 gotcha, but then the second button should have been call Do not Trash 😃.

@Shakil-Shahadat @mifi Please let's also include an option to clear all segments as well, either in file menu or trash dialogue #537.

mifi pushed a commit that referenced this issue Jan 18, 2021
Button titles simplified and made more recognizable. See #541 no 3.
@mifi
Copy link
Owner

mifi commented Jan 18, 2021

@Shakil-Shahadat thanks. I'm puzzled why the text is not left-aligned on your computer. Is it windows/linux?

@Shakil-Shahadat
Copy link
Contributor Author

I'm on Windows 8.1.

mifi added a commit that referenced this issue Jan 21, 2021
* Changes the default selection on Trash Dialog

Changed the default option to a safer one. See #541 point 2.

* make trash button default

Co-authored-by: Mikael Finstad <finstaden@gmail.com>
@mifi
Copy link
Owner

mifi commented Jan 21, 2021

@Shakil-Shahadat Thanks for reporting the alignment issue. for some reason the order of css is different in your build than mine.
I will fix it with !important
sweetalert2/sweetalert2#1525

Oh and if you cannot build on windows, then please report an issue, I'm happy to assist.

@markusd1984 clear all segments is under edit menu already

mifi added a commit that referenced this issue Jan 23, 2021
@Shakil-Shahadat
Copy link
Contributor Author

Everything is working fine.

I have managed to build but there were some warnings. I'll let you know when I test again.

Thank you so much.

@dandv
Copy link

dandv commented May 20, 2021

I still see -html5ified-dummy.mkv files after exporting and closing LossLess cut. What are those files for?

I guess my expectation was to not see any temporary files in the source directory after I close the application. Maybe create them in $TMP? The .csv is somewhat intuitive (it's the project file, I guess), but the HTML5 file I haven't seen until now. Maybe because the video wasn't natively supported? Anyway, seems like a pretty technical detail to the casual user.

@Shakil-Shahadat
Copy link
Contributor Author

@dandv You need to click the trash button to clean those files. They are needed to play video files.

@mattack1
Copy link

I really wish it would just put these files in /tmp or automatically delete them on quit (with a pref for that functionality I guess).

@mifi
Copy link
Owner

mifi commented Jul 25, 2021

The dummy files are used in order to preview files that originally are not supported by the player.
The reason why these dummy files are not deleted is because they may take some time to generate. so if the user re-opens the same file later they shouldn't have to wait for that file to regenerate.
The reason why they are stored in the same place as output files, is that the may take a lot of space if the input video is large. Then it can overwhelm the primary hard drive of most computers (temp directory). I underrstand that many people edit videos on external drives, because they have more space there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants