Bugzilla – Full Text Bug Listing
| Summary: | Update Generator UI to indicate which extensions have been applied to the workspace | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Brian Svihovec <svihovec> | ||||||
| Component: | EDT | Assignee: | pf yu <pfyu> | ||||||
| Status: | REOPENED --- | QA Contact: | |||||||
| Severity: | enhancement | ||||||||
| Priority: | P3 | CC: | chenzhh, jeffdouglas, smythew, tdramsey, tww | ||||||
| Version: | unspecified | ||||||||
| Target Milestone: | --- | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Whiteboard: | Extensibility_Gen | ||||||||
| Attachments: |
|
||||||||
|
Description
Brian Svihovec
I have marked this as I2, but I think it can be implemented in I3 if necessary. I think we want to implement this in the .8 release so that users can easily see what extensions have been added (i.e. IBMi). Created attachment 210630 [details]
fix
Hi, Brian Please help to review the implementation. Thanks Created attachment 210631 [details]
screen shot
I have a couple of questions: 1) does the list allow for multiple entries to be presented, because there can be many contributors for a specific provider 2) does the list also scan the "requires" settings, and bring in all of the contributors for the required ids as well for example: If a user created their own generator, that extended JavaGen, then they would need to create the extension point for their contributor, and that new contributor would have to include the requires id of JavaGenerationContributor as well. This means that their entire list to be displayed in the UI, would include all contributors for their extension PLUS all contributors for JavaHen, etc. Hi Jeff, For 1), yes, it can support multiple entries For 2), I use AbstractGenerator.determineContributions(id, contributionsUsed); method you have provide to get the contributors, I find that in the implementation of the method, you have scan the "requires" settings, and bring in all of the contributors for the required ids as well If no more question, I will check in the code tomorrow. finished After reviewing the FVT scenario for this line item, I believe we may need to update the UI. In the current screen capture, there are 3 columns for a Contributor: * id * class * provider While these accurately represent the information in the contributor extension point, I'm not sure a user will need to know any of this information. I would think that the ID should be replaced with a human readable Name, which would need to be provided in the extension point. I'm not sure the class name is useful? I believe the Provider basically duplicates the information in the selected tab (i.e. The Java Generator tab's provider shows as "org.eclipse.edt.ide.gen.JavaGenProvider")? In general, I think a user should be able to select a generator and see a list of named contributions, and possibly a version number. I realize that the extension point doesn't have a name or version number right now, but we may need to add these. We may be able to use the version number from the contributions plug-in. Reopen this enhance base on comment 8, need to change the display contributor info to: * name * version These info need Jeff to add these extension feilds to let user to input. This issue will require more discussion between Tim and Will before any changes can be made. We may decide to hide this UI addition in .8 I2 and revisit this in I3. More information will be posted soon. Have hide this in I2 and plan it to I3 after Tim and Will has made decision Hi, Tim and Will, Please make a quick decision so that I can finish it in I3 While we are making good progress on our extensibility design for EDT, I don't think we have reviewed the core design in enough detail to determine what the UI should look like at this time. I think we should move this work item to the Future list, and we can revisit this work in the next release. set the target date to future base on Comment 13 Setting the Target Milestone correctly for all bugs (set to Future in this case). |