Bugzilla – Full Text Bug Listing
| Summary: | [outline] Support StyledText in outline | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Technology] DLTK | Reporter: | Simon Bernard <sbernard> | ||||||||||||||
| Component: | Common | Assignee: | dltk.common-inbox <dltk.common-inbox> | ||||||||||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||||||||||
| Severity: | enhancement | ||||||||||||||||
| Priority: | P3 | CC: | bruno.do.medeiros, contact, kaloyan, zulus | ||||||||||||||
| Version: | 5.0 | ||||||||||||||||
| Target Milestone: | 5.2 | ||||||||||||||||
| Hardware: | PC | ||||||||||||||||
| OS: | Windows 7 | ||||||||||||||||
| See Also: |
https://git.eclipse.org/r/36425 https://git.eclipse.org/c/dltk/org.eclipse.dltk.core.git/commit/?id=d473c2a75363ace72dc833bf66e60655cebd286b |
||||||||||||||||
| Whiteboard: | |||||||||||||||||
| Bug Depends on: | |||||||||||||||||
| Bug Blocks: | 413518 | ||||||||||||||||
| Attachments: |
|
||||||||||||||||
|
Description
Simon Bernard
Created attachment 233624 [details]
a patch for this bug
A patch inspired from JDT.
Created attachment 233626 [details]
a additional patch
an additional patch which replace all references of deprecated DecoratingModelLabelProvider by the new StyledDecoratingModelLabelProvider (without API break)
Created attachment 233697 [details]
support of styledString in quick ouline.
this patch add the support of styled string in quick outline.
Created attachment 233698 [details]
support of styledString in content assist.
This patch add support of styled string in content assist.
Created attachment 233889 [details]
a path for this bug.
I fix a regression I added on ScriptUILabelProvider.
Created attachment 233890 [details]
support of styledString in viewpart (scriptexplorer,...)
I fix a regression on package name.
Hi, I get no feedback about my patches ? Do you think it was able to integrated it for the next release ? ping ? pushed on gerrit : https://git.eclipse.org/r/#/c/36425/ @Kaloyan is this possible to push this into 5.2? Simon, I apologize for the really long delay. I was able to test the suggested patch together with the one in bug 413518. It looks OK. There are just a few minor things to clean up in the code (see comments in Gerrit) and we are good to go. I do the changes. I hope it's ok now. :) Gerrit change https://git.eclipse.org/r/36425 was merged to [master]. Commit: http://git.eclipse.org/c/dltk/org.eclipse.dltk.core.git/commit/?id=d473c2a75363ace72dc833bf66e60655cebd286b Merged. Thanks for the contribution! Kaloyan, will it be available in the next eclipse milestones (Mars M7) ? Yes. Closing. |