If importing a project failed because its exist, ask the user to open/jump to the project

[laeubi]

Consider the following situation:

1. One has imported a project X in the project but closed it
2. No one uses the Wizard "Import existing projects" or right click in the Git Repositories View and choose "Import Projects ...)
3. Try to import project X and get an error that it is already imported

This has bad user experience as the user obviously would not have tried to import it if he was aware of that fact and the dialog does not really help here.

Instead there should be the following options:

1. The project is open, then ask the user to jump to the project (e.g. in Package/Project Explorer)
2. If the project is closed, ask the user to open the project

[ishayankoo001]

Hi, I would like to start working on this if possible. This would be my first open-source contribution and I'm excited to learn. Could you please also recommend where I should start looking?

[laeubi]

Just setup a dev-environment and start exploring / coding!

If you have questions let us know, either here, or show a PR with your current state.

It is usually a good starting point to search for a String in the UI inside the IDE and look where the function is located, the add a Breakpoint and further explore the surrounding code.

[HannesWell]

Just setup a dev-environment and start exploring / coding!

Yes that's a good start and if you need a bit more guidance you can have a look at

• https://github.com/eclipse-ide

[ishayankoo001]

Hi, thank you for your helpful insights. I have a few questions, and I apologize if they are basic.
It's my current understanding that the code for the import wizard, specifically SmartImportRootWizardPage.java and its parent SmartImportWizard.java, is located inside the org.eclipse.ui.ide plug-in, which is part of the platform.ui component.
For my specific task of opening an already-imported project, it seems the core logic to do this (like methods on the IProject interface from org.eclipse.core.resources) already exist. Does this mean the new logic would be contained entirely within SmartImportRootWizardPage.java, and I won't need to change files in other plug-ins?
My main confusion is that the issue was filed against the general "Platform" component. I just want to confirm my understanding: Is it correct that "Platform" is the high-level component, and the file I actually need to modify for this feature is indeed SmartImportRootWizardPage.java, which is inside platform.ui?

[laeubi]

platform and platform.ui actually belong together and are closely coupled. So we can move it there or jut keep it here it does not matter much, in the worst case you even need to modify both places...