| Summary: | user can DnD url from web browser into Eclipse to start MPC install sequence | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Technology] MPC | Reporter: | Benjamin Muskalla <b.muskalla> | ||||||||
| Component: | wizard | Assignee: | Benjamin Muskalla <b.muskalla> | ||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||
| Severity: | enhancement | ||||||||||
| Priority: | P3 | CC: | greensopinion, ian.skerrett, nathan | ||||||||
| Version: | unspecified | ||||||||||
| Target Milestone: | 1.1 | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Benjamin Muskalla
Created attachment 189422 [details] draft patch David, can we have a quick look at this together? Open items: * what to do if we don't know the marketplace? in my POV, I'd just fail for now. This could be improved once we have server-side catalog listings that could provide us with the right catalog. * URL format: My idea is that it can be part of every url as part of the query. This would allow marketplaces to always use these links to point to some descriptive page if clicked (eg http://marketplace.eclipse.org/marketplace-client-intro) and use the same url for DnD into Eclipse. Is there any reason on the current marketplace to use Forms with GET instead of raw links? If we can agree on one form, we could easily merge this with MPBrowserIntegration. Created attachment 189715 [details]
final patch
David, attached is the final patch for this. I iterated over our discussed items and merged the implementations of MPBrowserIntegration and the Drop Adapter. The idea to check for the URL early is not possible as SWT doesn't propagate the data until the drop event occurs.
Created attachment 189716 [details]
sample html page
Here is a small html page with different link types that can be dropped on Eclipse to start the installation process.
Please note that DnD from the Install button on the current marketplace is not possible due to the usage of the form tag. I recommend changing this to a raw link as shown in the html snippet, that solution is also backwards-compatible. Will talk to Nathan about this. Comment on attachment 189715 [details]
final patch
patch applied, with fixes to copyright header, and MPC drop adapter is registered with a UI job instead of using Display.asyncExec
Tested, looks great. Note that in my testing only certain areas of Eclipse were able to accept the drop. Benjamin please resolve if there's nothing left to do here. Thanks. Yes, that's a limitation of the operation system. For example on Linux, you cannot drop anything on the MenuBar of the Shell. Nothing we can do here with the "limitations" (SWT) we have. |