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

Fix an issue where a file is deselected when returning to the folder compare window after opening the file compare window by double-clicking the file in the folder compare window. #857

Merged
merged 1 commit into from Jul 4, 2021

Conversation

tjmprm77
Copy link
Contributor

@tjmprm77 tjmprm77 commented Jul 4, 2021

Sometimes a file is deselected when returning to the folder compare window after opening the file compare window by double-clicking the file in the folder compare window.

For example, it occurs by the following procedure.
Step 0. Set as follows.

  • The "Include subfolders" option is on.
  • The display of the message box "The selected files are identical." is enabled.

Step 1. Compare directories with the same three files.
Step 2. Adjust the window size to just fit the three files.
Step 3. Double-click on the third file. This open a file compare window with the message "The selected files are identical."
Step 4. Close the message box and select the folder compare window. At this time, the third file is deselected.
diff

It is a bit inconvenient because we lose track of which file had been checked when checking the differences between the files in the folder one by one, if the file is deselected when we return to the folder compare window.

Regarding this behavior, I suspect it is not good that CListView::OnLButtonDblClk() is executed while the focus is out of the folder compare window by opening the file compare window.

With this PR, CDirView::OnLButtonDblClk() now executes CListView::OnLButtonDblClk() only when it is in focus.

…compare window after opening the file compare window by double-clicking the file in the folder compare window.

- CDirView::OnLButtonDblClk() now executes CListView::OnLButtonDblClk() only when it is in focus.
@sdottaka sdottaka merged commit dae9035 into WinMerge:master Jul 4, 2021
@sdottaka
Copy link
Member

sdottaka commented Jul 4, 2021

Thank you!

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

Successfully merging this pull request may close these issues.

None yet

2 participants