Bugzilla – Full Text Bug Listing
| Summary: | DecoratedField unusable | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [RT] RAP | Reporter: | Thomas Schindl <tom.schindl> | ||||
| Component: | JFace | Assignee: | Project Inbox <rap-inbox> | ||||
| Status: | RESOLVED WONTFIX | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | ||||||
| Version: | unspecified | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Mac OS X - Carbon (unsup.) | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
I will close it as WONTFIX as DecoratedField is deprecated and ControlDecoration is available since 1.3M4 ( see Bug 251689 ). |
Created attachment 140628 [details] Patch The current implementation of DecoratedField is unuseable because it pops up a big div for every DecoratedField when adding a Decoration. This is because of the code in showHoverText(). The shell created doesn't even display the error information (the original JFace-Paint-Code is commented naturally). In the attached patch i also added a new feature which hides the decoration when no error/warning/... is displayed (though not completed yet). I'd suggest to at least comment the shell creation in showHoverText in the next minor release and discuss whether you are willing to add the new feature to RAP to make it at least in a none decoration case more compatible with ControlDecoration (the patch has 2 fix me positions I'd work on if you agree that this is a good thing) when maintaining a RAP and RCP code base.