| Summary: | Use FilteredTree in new feature project wizard | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] PDE | Reporter: | Lars Vogel <Lars.Vogel> | ||||||
| Component: | UI | Assignee: | Martin Karpisek <martin.karpisek> | ||||||
| Status: | VERIFIED FIXED | QA Contact: | |||||||
| Severity: | enhancement | ||||||||
| Priority: | P3 | CC: | cpuidle, curtis.windatt.public, d.spiridenok, Lars.Vogel, markus.kell.r, martin.karpisek, Vikas.Chandra | ||||||
| Version: | 3.4.1 | Keywords: | noteworthy | ||||||
| Target Milestone: | 4.7 M4 | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| See Also: |
https://git.eclipse.org/r/82317 https://git.eclipse.org/r/82322 https://git.eclipse.org/r/82418 https://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=4c1edad836207649d0dd9fbeee29de301c7d2a86 https://git.eclipse.org/r/85902 https://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=238c8572283f26b393c9891bf38d17e4227381ee |
||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
|
Description
Lars Vogel
Created attachment 112531 [details]
Feature Dialog
tagging bugday. Thanks for the suggestion. Created attachment 144594 [details]
Patch for PluginListPage
Patch adds a filter field and the necessary functionality to PluginListPage of new feature wizard...
Thanks for the patch, will review. This patch does not add a proper filter to the tree. Filtering breaks the current selection, the selection count, etc. The layout of the page is broken as well. It is not easy to get a filtered checkbox list/tree. There are many problems with the one in the launch config and they are poor performance wise. That being said, the PDE implementation FilteredCheckboxTree can work. *** Bug 449673 has been marked as a duplicate of this bug. *** *** Bug 283143 has been marked as a duplicate of this bug. *** New Gerrit change created: https://git.eclipse.org/r/82317 New Gerrit change created: https://git.eclipse.org/r/82322 New Gerrit change created: https://git.eclipse.org/r/82418 Gerrit change https://git.eclipse.org/r/82317 was merged to [master]. Commit: http://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=4c1edad836207649d0dd9fbeee29de301c7d2a86 Thanks Martin. Could you add this also to the N&N M4? New Gerrit change created: https://git.eclipse.org/r/85902 Gerrit change https://git.eclipse.org/r/85902 was merged to [master]. Commit: http://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=238c8572283f26b393c9891bf38d17e4227381ee Lars, can you verify this fix? verified on Version: Oxygen (4.7) Build id: I20161205-2000 |