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

New close actions are not working #17

Closed
funfried opened this issue Jun 20, 2023 · 4 comments
Closed

New close actions are not working #17

funfried opened this issue Jun 20, 2023 · 4 comments
Assignees
Labels
Milestone

Comments

@funfried
Copy link
Owner

It seems the new project related close actions as well as the VCS related close action are not working correctly. The VCS close action just closes every tab and the project related actions seem to close too many tabs.

@funfried funfried added the bug label Jun 20, 2023
@funfried funfried added this to the 1.0.4 milestone Jun 20, 2023
@funfried funfried self-assigned this Jun 20, 2023
XCame pushed a commit to XCame/nb-editor-close-left-right that referenced this issue Jul 17, 2023
The problem was, that later checks returning false for isModified() would overwrite earlier "true"s.
e.g. if gitModified was true, svnModified was null and hgModified was false, the file would be closed
because of the hgModified false passed (hgModified != null && !hgModified) and closed the file although
gitModified was true.

Signed-off-by: Fabian Kreuzkam <fabian@kreuzkam.com>
@ggrachdev
Copy link

Good afternoon. The changes don't work, at least with Subversion. There are tabs that have not been changed in VSC (the black color of the tab text) and changed (the blue color of the tab text), the plugin closes all files (including blue ones, although they are not marked). Do I understand the implemented functionality correctly, that it shouldn't be like this? Thanks!

@funfried
Copy link
Owner Author

funfried commented Aug 4, 2023

Yes, true, I added a bug there and there is already a PR from @XCame but I couldn't merge it yet, I will publish a bug fix release within the next days. Stay tuned.

Cheers,
Fabian

funfried added a commit that referenced this issue Aug 7, 2023
funfried added a commit that referenced this issue Aug 7, 2023
Fixed build issue and closing commited tabs
@funfried
Copy link
Owner Author

funfried commented Aug 9, 2023

Should be fixed in the current SNAPSHOT version and therefore within the up coming release.

@funfried funfried closed this as completed Aug 9, 2023
@ggrachdev
Copy link

Checked at work. So far, at least. Thank you!

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

No branches or pull requests

2 participants