This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 419639 - [JFace] Remove outdated JFace Snippets
Summary: [JFace] Remove outdated JFace Snippets
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.4 M7   Edit
Assignee: Lars Vogel CLA
QA Contact: Wojciech Sudol CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 420779
  Show dependency tree
 
Reported: 2013-10-16 17:52 EDT by Lars Vogel CLA
Modified: 2014-03-18 07:36 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2013-10-16 17:52:30 EDT
Currently the JFace Snippets contain several examples with outdated API.

I suggest to remove them from current master to make is simpler for new people to get started.

I currently see the following candidates

Snippet022TableViewerRefreshNoScroll
Snippet033CellEditorPerRowPre33
Snippet058CellNavigationIn34.java
Snippet059CellNavigationIn33.java

If we remove them, we need also to update the wiki.
Comment 1 Lars Vogel CLA 2013-10-16 18:23:50 EDT
Snippet023TreeViewerCustomTooltips
Comment 2 Lars Vogel CLA 2013-11-12 17:12:11 EST
If there are no objections I remove these snippets soon. They will still be accessible via Git for those interested in ancient code. ;-)
Comment 3 Lars Vogel CLA 2014-02-12 18:08:49 EST
Snippet032TableTreeViewer demonstrates the deprecated TableTreeViewer

https://git.eclipse.org/r/21924
Comment 4 Lars Vogel CLA 2014-03-18 05:45:48 EDT
(In reply to Lars Vogel from comment #3)
> Snippet032TableTreeViewer demonstrates the deprecated TableTreeViewer
> 
> https://git.eclipse.org/r/21924

Fixed with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=3a2d6a06108dbae2698aa08d6f91d0e2d8330de3
Comment 5 Lars Vogel CLA 2014-03-18 05:53:21 EDT
Snippet042ToolTipSupportFor32API
Comment 7 Lars Vogel CLA 2014-03-18 06:03:46 EDT
Also update the JFace Wiki with the snippets
Comment 8 Markus Keller CLA 2014-03-18 07:17:42 EDT
How can this bug be VERIFIED if it leaves a compile error?

Fixed with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=3507dde1d64ac8aa38804e2c728b5254b7135c5a
Comment 9 Lars Vogel CLA 2014-03-18 07:36:56 EDT
(In reply to Markus Keller from comment #8)
> How can this bug be VERIFIED if it leaves a compile error?
> 
> Fixed with
> http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/
> ?id=3507dde1d64ac8aa38804e2c728b5254b7135c5a

Thanks Markus for the follow-up, I forgot to include one change in the Gerrit review.