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

3-way merge editor #12653

Open
Tracked by #13192
airaketa opened this issue Jun 27, 2023 · 6 comments
Open
Tracked by #13192

3-way merge editor #12653

airaketa opened this issue Jun 27, 2023 · 6 comments
Labels
enhancement issues that are enhancements to current functionality - nice to haves git issues related to git help wanted issues meant to be picked up, require help

Comments

@airaketa
Copy link

Hello,
I wonder if there are any plans to implement 3 way merge editor in Theia?

One of the approaches would be to integrate 3-way merge editor from VS Code but I'm not sure how hard it would be - it is part of a vscode.git plugin which, as far as I understand, will overlap with own implementation (git package in Theia). I also tried to superficially test it by installing vscode.git plugin but it seems to not function.
Pull request introducing 3-way merge editor for VS Code is located here: microsoft/vscode#150391

Feature Description:

3-way merge editor (one can take VS Code 3-way merge editor as a reference)

@vince-fugnitto vince-fugnitto added enhancement issues that are enhancements to current functionality - nice to haves git issues related to git labels Jun 27, 2023
@vince-fugnitto
Copy link
Member

@airaketa there are no plans as of yet to include the 3 way merge editor in the framework so if interested you can certainly contribute 👍 There are changes required in the framework to get the feature to work as the changes are not all local to the git builtin.

For additional information, the @theia/git extension precedes support for the vscode.git builtin for the framework, if interested in using vscode.git then you can drop @theia/git entirely in your application, they should be consumed exclusively.

@Ricbet
Copy link

Ricbet commented Aug 25, 2023

Hi, @vince-fugnitto, I'm @Ricbet, the core developer of opensumi.
The 3-way merge editor feature has been implemented in our opensumi framework.
However, unlike the mode in vscode, it uses a three-panel layout with left, center, and right columns to resolve conflicts. like:

image

It was inspired by visual studio, but we based it on the monaco editor.
I'm not sure if the official team or users of Theia would accept this type of interaction. If they do accept it, I would be happy to implement a similar 3-way merge editor in Theia.

Here is the functionality implemented in opensumi, you can refer to it:
Related issue: opensumi/core#1930
Related pull request: opensumi/core#1960

@vince-fugnitto
Copy link
Member

@Ricbet thank you for reaching out!

The 3-way merge editor would definitely be a very nice contribution to the framework, especially given how the vscode.git permits the option through the git.mergeEditor preference. Please feel free to submit a pull-request!

@tsmaeder tsmaeder mentioned this issue Apr 10, 2024
55 tasks
@pisv
Copy link
Contributor

pisv commented Jul 23, 2024

Hi, just wanted to check what is the current state of this issue? I could start working on it, if help is still wanted.

@msujew msujew added the help wanted issues meant to be picked up, require help label Jul 23, 2024
@msujew
Copy link
Member

msujew commented Jul 23, 2024

@pisv I don't think anyone else is working on this right now. Feel free to pick it up :)

@pisv
Copy link
Contributor

pisv commented Jul 23, 2024

OK :) I will pick it up, then. Thank you for so prompt reply!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement issues that are enhancements to current functionality - nice to haves git issues related to git help wanted issues meant to be picked up, require help
Projects
None yet
Development

No branches or pull requests

5 participants