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

Clear-compare closes original in main view #48

Closed
xylographe opened this issue Apr 12, 2016 · 7 comments
Closed

Clear-compare closes original in main view #48

xylographe opened this issue Apr 12, 2016 · 7 comments

Comments

@xylographe
Copy link
Contributor

CP 1d3e95a
NPP 6.9.1
Win7SP1[64]

  • Open two files in main view: (A, B)
  • Clone B to sub view: (A, B | B)
  • Select A in main view
  • Trigger A-B compare
  • Clear compare

Result: diff marks are cleared and B is closed in the main view: (A | B)
Expected: diff marks are cleared and no further action: (A B | B)

No big deal, as I would normally close B in the sub view (I can now move B to the main view instead), but I thought I should mention it anyway.

@pnedev pnedev added the bug label Apr 12, 2016
@pnedev
Copy link
Owner

pnedev commented Apr 12, 2016

I'll have a look soon, thanks for reporting.
Sounds like a N++ issue at first look but needs further investigation.

BR

@Yaron10
Copy link

Yaron10 commented Apr 13, 2016

Hello xylographe,

Thanks for that.

Hello Pavel,

Thank you for your work. :)

BR

@pnedev
Copy link
Owner

pnedev commented Apr 25, 2016

Hello @xylographe ,

N++ cloned file view is not correct after the "Clone" operation.
If you clone file A from main view, the cloned A file in the second view has buffer Id corresponding to the main view.

Shortly put - it is a N++ issue as I suspected.
I cannot think of a proper workaround for this - N++ needs to be fixed in this case.
It was FYI, Thanks.

BR

@xylographe
Copy link
Contributor Author

Thank you, Pavel, for looking into and explaining this issue.
I won't put my money on a fix in NPP. :) Fortunately, there are alternatives.

@pnedev
Copy link
Owner

pnedev commented Apr 26, 2016

Hello @xylographe ,

I'll reopen the issue just to have it logged as a known problem. Thanks.

BR

@pnedev pnedev reopened this Apr 26, 2016
This was referenced May 19, 2016
@pnedev pnedev added this to the Never milestone May 20, 2016
@pnedev
Copy link
Owner

pnedev commented May 25, 2016

Hello,

Now that we have "Known issues" list I'll close this one.
Thanks.

BR

@pnedev pnedev closed this as completed May 25, 2016
pnedev added a commit that referenced this issue Oct 20, 2016
Updated sqlite libs to v3.15.0
@Yaron10
Copy link

Yaron10 commented Sep 17, 2018

Hello,

As of now - the file is closed on Compare (i.e. before Clear).

BR

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

No branches or pull requests

3 participants