[go: nahoru, domu]

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

Merge option broken in newer versions "Please open at least 2 files to merge, then try again" #1316

Closed
2 of 4 tasks
deton24 opened this issue Sep 16, 2022 · 1 comment
Closed
2 of 4 tasks

Comments

@deton24
Copy link
deton24 commented Sep 16, 2022

I have a lot of issues to go through, so in order to make it easier for me to help you, I ask that you please try these things first

Operating System

Windows 10

Steps to reproduce

  1. Extract LosslessCut-win-x64.7z
  2. Open the app
  3. Go to Tools/Merge/Concatenate files

Expected behavior

There should be a window to choose files to merge like it is in e.g. 3.4.2 version. Now only error appears.

Well, I tried to do this the other way round, and open two files at the same time, but it either didn't work or wasn't intuitive enough to perform like it was before.

Actual behavior

Error appears instead of a window to chose files to merge ("Please open at least 2 files to merge, then try again")

Share log

No response

@mifi
Copy link
Owner
mifi commented Sep 29, 2022

I will change the logic so that it will instead show the Open files-dialog if the Merge menu item is clicked without any files already open

@mifi mifi closed this as completed Sep 29, 2022
mifi added a commit that referenced this issue Sep 29, 2022
show the Open files-dialog if the Merge menu item is clicked without any files already open
had to rearrange a lot of code due to deps

fixes #1316
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

2 participants