Bug 260061 - [Snippets] Example Snippet how to implement native checkboxes
Summary: [Snippets] Example Snippet how to implement native checkboxes
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 3.5 RC1   Edit
Assignee: Thomas Schindl CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-06 11:31 EST by Thomas Schindl CLA
Modified: 2016-03-11 01:50 EST (History)
10 users (show)

See Also:


Attachments
Patch with the implementation (13.41 KB, patch)
2009-01-06 11:31 EST, Thomas Schindl CLA
no flags Details | Diff
Revised (14.22 KB, patch)
2009-01-07 12:15 EST, Thomas Schindl CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Schindl CLA 2009-01-06 11:31:05 EST
Created attachment 121646 [details]
Patch with the implementation

An example combining informations and tricks implementing a:
- native looking multi checkbox viewer
- supports tabbing
- ...

The intial work used is the one from my blog including comments from people on the blog so maybe we need a CQ.
Comment 1 Thomas Schindl CLA 2009-01-06 11:38:39 EST
The initial code was always available from my companies public SVN repo a released there [1] and the comments [2] and [3].

We should also test on Win32 and GTK to see if the adjustments in the OwnerDrawLabelProvider are appropriate there.

Boris do we need a CQ for this? I was told by Wayne that if the code was made public in a public area other than Eclipse (my own SVN-Repo) that we need a CQ (we also have contributors here [2] and [3]).

[1]http://publicsvn.bestsolution.at/repos/java/at.bestsolution.jface/branches/3_3_support/src/at/bestsolution/jface/viewers/EmulatedNativeCheckBoxLabelProvider.java
[2]http://tom-eclipse-dev.blogspot.com/2007/01/tableviewers-and-nativelooking.html#c5864843583641560752
[3]http://tom-eclipse-dev.blogspot.com/2007/01/tableviewers-and-nativelooking.html#c8984638995490184927
Comment 2 Boris Bokowski CLA 2009-01-06 12:43:16 EST
(In reply to comment #1)
> Boris do we need a CQ for this? I was told by Wayne that if the code was made
> public in a public area other than Eclipse (my own SVN-Repo) that we need a CQ
> (we also have contributors here [2] and [3]).

Yes, you should file a CQ through portal.eclipse.org. Let me know if you need help.
Comment 3 Thomas Schindl CLA 2009-01-06 15:14:03 EST
The CQ [1] is filled and I contact the commenter on my blog to ask him for confirmation. 

[1]https://dev.eclipse.org/ipzilla/show_bug.cgi?id=2925
Comment 4 Thomas Schindl CLA 2009-01-07 12:15:46 EST
Created attachment 121831 [details]
Revised
Comment 5 Florian Potschka CLA 2009-01-09 19:10:37 EST
My comment on Toms blog contributed some improvements to his code. These improvements were written solely by me. So of course you can add this code to the Eclipse JFace Snippet Collection. Perhaps you could add a short comment that names me as a contributor, that would be great.
Comment 6 Alexander Ljungberg CLA 2009-01-11 00:10:49 EST
I did contribute a few lines. Is there anything in particular I need to do here?
Comment 7 Sharon Corbett CLA 2009-01-13 09:40:34 EST
Hi Alexander; if you could just confirm your answers to the following question on this bug would be great:

1.  You authored the lines of code you contributed 100%
2.  You have the rights to donate the content to Eclipse
3.  You are submitting them to Eclipse under the EPL

Thanks,
Sharon Corbett
Eclipse Intellectual Property
Comment 8 Alexander Ljungberg CLA 2009-01-13 14:40:16 EST
Absolutely. 

1. I wrote the code that I contributed to this snippet 100%.
2. I have the rights to donate the content to Eclipse.
3. I am submitting the contribution to Eclipse under the EPL.

Let me know if there is anything else you need.
Comment 9 Al B CLA 2009-01-21 12:31:27 EST
FYI: the makeShot method is currently leaking since it needs to dispose the variables image and greenScreen.
Comment 10 Boris Bokowski CLA 2009-05-07 16:05:12 EDT
btw, the CQ has been approved a while ago.
Comment 11 Thomas Schindl CLA 2009-05-08 13:09:21 EDT
Released with some more tweaks for OS-X to CVS. We should discuss in whether we should probably push the Native-Combo-CellEditor implementation up to JFace.
Comment 12 Thomas Schindl CLA 2009-05-14 12:35:49 EDT
this is not part of the release but the code is in the repo
Comment 13 Doug CLA 2009-07-29 07:38:03 EDT
I'm voting for this too be pushed into JFace proper. Its a neat solution to a common problem.
Comment 14 Florian Potschka CLA 2009-07-29 18:08:18 EDT
(In reply to comment #13)
> I'm voting for this too be pushed into JFace proper. Its a neat solution to a
> common problem.
> 

+1

Comment 15 Thomas Schindl CLA 2009-07-29 19:06:13 EDT
if you want that you'll need to file a new bug - I'm a bit worried on how the screenshot is take so we need to discuss with SWT-Team
Comment 16 Doug CLA 2009-07-30 03:53:49 EDT
(In reply to comment #15)
> if you want that you'll need to file a new bug - I'm a bit worried on how the
> screenshot is take so we need to discuss with SWT-Team
> 

I raised bug 285121 - please vote for that one.