[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [nebula-dev] Proposed New Widget
|
Keep it as small as possible so I'd expose the Table-Class.
Tom
Marty Jones schrieb:
> Tom,
>
> Thanks for the input. I have the TableCombo now separated into a SWT
> TableCombo class and a JFace TableComboViewer class.
>
> Both implementations are working but I am not comfortable with having
> the TableCombo class expose it's Table reference as public so that the
> SWT version can create new TableItem rows and the JFace version to be
> able to create new TableItem's as well.
>
> I thought about wrapping the TableItem object and just creating a new
> TableItem when the new WrapperTableItem is created but that seems like
> overkill. Do you have a opinion as to whether or not the Table
> reference is ok being public or do you think I should find a solution
> where the Table reference is not exposed?
>
> Marty
>
> On Wed, Oct 7, 2009 at 4:04 AM, Tom Schindl <listom@xxxxxxxxxxxxxxx
> <mailto:listom@xxxxxxxxxxxxxxx>> wrote:
>
> Hi Marty,
>
> The less the better but we also allow widgets to use Java5 if they
> really need it but Java5 is the maximum because there are still a lot of
> users (especially on OS-X who don't use Java6).
>
> So if there's a real benefit for the user than use Java5 else stick to
> JDK 1.4 or even better 1.3 :-)
>
> Tom
>
> Marty Jones schrieb:
> > I have a couple of questions.
> >
> > What version of the JDK should I code the widget against?
> >
> > Can we use Generics or the enhanced for loop?
> >
> > What about using Lists instead of arrays?
> >
> > Is there any guidelines to what JDK features you should not use?
> >
> >
> > Thanks,
> >
> > Marty
> >
> >
> > On Tue, Oct 6, 2009 at 10:53 AM, Tom Schindl
> <listom@xxxxxxxxxxxxxxx <mailto:listom@xxxxxxxxxxxxxxx>
> > <mailto:listom@xxxxxxxxxxxxxxx <mailto:listom@xxxxxxxxxxxxxxx>>>
> wrote:
> >
> > Hi,
> >
> > Yes but take a look at the CTreeCombo sources (they never got
> part of
> > Nebula because I'm lacking time to maintain one more thing).
> >
> > The sources are here:
> >
> http://publicsvn.bestsolution.at/repos/java/examples/EMF-Databinding/at.bestsolution.soccer.ui.common/src/at/bestsolution/soccer/ui/common/widgets/
> >
> http://publicsvn.bestsolution.at/repos/java/examples/EMF-Databinding/at.bestsolution.soccer.ui.common/src/at/bestsolution/soccer/ui/common/viewers/
> >
> > I'm going to file a CQ later and for your bug and you can do the
> > refactoring probably inside Nebula.
> >
> > Tom
> >
> > Marty Jones schrieb:
> > > Gotcha. I will take a look at the Grid and RadioGroup
> widgets to see
> > > how they are configured to use optional dependencies.
> > >
> > > Do you guys think that if I can get my widget to work under
> > straight SWT
> > > as well as JFace that it would qualify to be added as a
> Nebula widget?
> > >
> > > On Tue, Oct 6, 2009 at 10:28 AM, Tom Schindl
> > <listom@xxxxxxxxxxxxxxx <mailto:listom@xxxxxxxxxxxxxxx>
> <mailto:listom@xxxxxxxxxxxxxxx <mailto:listom@xxxxxxxxxxxxxxx>>
> > > <mailto:listom@xxxxxxxxxxxxxxx
> <mailto:listom@xxxxxxxxxxxxxxx> <mailto:listom@xxxxxxxxxxxxxxx
> <mailto:listom@xxxxxxxxxxxxxxx>>>>
> > wrote:
> > >
> > > Hi Marty,
> > >
> > > I don't think we want different bundles but one bundle
> with an
> > optional
> > > dependency but didn't set up extra rules for this (at
> least the
> > > Nebula-Grid does it this way and the RadioGroup-Widget
> as well).
> > >
> > > Tom
> > >
> > > Marty Jones schrieb:
> > > > Emil,
> > > >
> > > > I am going to see if I can separate out the JFace version
> > and create
> > > > a straight SWT module as well. Do you have any
> examples of
> > where this
> > > > was done with one of the Nebula Widgets? I am trying
> to find a
> > > example
> > > > of how this has been done in the past.
> > > >
> > > > Marty
> > > >
> > > > On Tue, Oct 6, 2009 at 2:55 AM, Emil Crumhorn
> > > <emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx> <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx>>
> > <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx> <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx>>>
> > > > <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx>
> > <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx>> <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx>
> > <mailto:emil.crumhorn@xxxxxxxxx
> <mailto:emil.crumhorn@xxxxxxxxx>>>>>
> > > wrote:
> > > >
> > > > Nice widget! I really like it. I see it depends on;
> > > >
> > > > org.eclipse.core.commands
> > > > org.eclipse.equinox.common
> > > >
> > > > and also JFace
> > > >
> > > > I'm not sure of the [current] policy of widgets in
> > Nebula, but it
> > > > used to be "pure SWT" and any JFace stuff had to
> be put as a
> > > > separate "module" of the "core" as an option to those
> > that want to
> > > > use JFace.
> > > >
> > > > Perhaps it's time to drop those rules, but,
> thought I'd
> > point it
> > > > out. Perhaps that can be worked around or are those
> > dependencies
> > > > critical (from a quick look it seems it's mostly
> the label
> > > providers
> > > > and intra-dependencies from JFace)?
> > > >
> > > > Cheers,
> > > > Emil
> > > >
> > > >
> > > >
> > > > On Tue, Oct 6, 2009 at 1:37 AM, Marty Jones
> > > <martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>
> <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>
> > <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>
> <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>>
> > > > <mailto:martybjones@xxxxxxxxx
> <mailto:martybjones@xxxxxxxxx>
> > <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>
> <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>
> > <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>>>>
> > > wrote:
> > > >
> > > > Chris,
> > > >
> > > > I have filed a bug with the TableCombo source
> attached (
> > > > *Bug 291432*
> > > >
> > <https://bugs.eclipse.org/bugs/show_bug.cgi?id=291432>). I
> > > > wasn't sure which component to choose so I
> chose "Core".
> > > >
> > > > You are correct in that the license is
> "Eclipse Public
> > > License".
> > > >
> > > > Thanks,
> > > >
> > > > Marty
> > > >
> > > >
> > > >
> > > > On Mon, Oct 5, 2009 at 4:43 PM, Chris Aniszczyk
> > > > <zx@xxxxxxxxxxxxxxxxx
> <mailto:zx@xxxxxxxxxxxxxxxxx> <mailto:zx@xxxxxxxxxxxxxxxxx
> <mailto:zx@xxxxxxxxxxxxxxxxx>>
> > <mailto:zx@xxxxxxxxxxxxxxxxx <mailto:zx@xxxxxxxxxxxxxxxxx>
> <mailto:zx@xxxxxxxxxxxxxxxxx <mailto:zx@xxxxxxxxxxxxxxxxx>>>
> > > <mailto:zx@xxxxxxxxxxxxxxxxx
> <mailto:zx@xxxxxxxxxxxxxxxxx> <mailto:zx@xxxxxxxxxxxxxxxxx
> <mailto:zx@xxxxxxxxxxxxxxxxx>>
> > <mailto:zx@xxxxxxxxxxxxxxxxx <mailto:zx@xxxxxxxxxxxxxxxxx>
> <mailto:zx@xxxxxxxxxxxxxxxxx <mailto:zx@xxxxxxxxxxxxxxxxx>>>>> wrote:
> > > >
> > > > On Mon, Oct 5, 2009 at 10:55 AM, Marty Jones
> > > > <martybjones@xxxxxxxxx
> <mailto:martybjones@xxxxxxxxx>
> > <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>
> <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>
> > <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>>
> > > <mailto:martybjones@xxxxxxxxx
> <mailto:martybjones@xxxxxxxxx> <mailto:martybjones@xxxxxxxxx
> <mailto:martybjones@xxxxxxxxx>>
> > <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>
> <mailto:martybjones@xxxxxxxxx <mailto:martybjones@xxxxxxxxx>>>>> wrote:
> > > >
> > > > Hello,
> > > >
> > > > My name is Marty Jones and I have
> created a
> > widget
> > > that
> > > > I call a TableCombo. This widget is based
> > upon the
> > > > CCombo widget but uses a Table for the
> drop down
> > > portion
> > > > of the combo instead of a List.
> > > >
> > > > My widget has the following features:
> > > >
> > > > o Supports displaying multiple
> columns of
> > > data in
> > > > the drop down portion of the combo.
> > > > o Supports displaying a header
> for the
> > drop down
> > > > portion of the combo.
> > > > o Supports images for each
> column that is
> > > > displayed within the drop down portion of
> > the combo.
> > > > o Supports custom Colors and
> Fonts for
> > each
> > > Row &
> > > > Column within the drop down portion of the
> > combo.
> > > > o Supports displaying a % of the
> full
> > drop down
> > > > portion of the combo. So you can specify
> > that the
> > > viewable
> > > > portion of the drop down
> will be
> > 75% of the
> > > > full width of the table and a Horizontal
> > scrollbar
> > > will
> > > > be displayed.
> > > >
> > > > I would like to submit this widget as a
> > candidate
> > > widget
> > > > for the Nebula Project.
> > > >
> > > > The source code can be downloaded from the
> > following
> > > > link:
> > www.modelsoftware.com/nebula/TableCombo.zip
> <http://www.modelsoftware.com/nebula/TableCombo.zip>
> > <http://www.modelsoftware.com/nebula/TableCombo.zip>
> > > <http://www.modelsoftware.com/nebula/TableCombo.zip>
> > > >
> > <http://www.modelsoftware.com/nebula/TableCombo.zip>
> > > >
> > > >
> > > > Very cool!
> > > >
> > > > The first step is to file a bug against Nebula
> > and attach
> > > > the source code:
> > > >
> > > https://bugs.eclipse.org/bugs/enter_bug.cgi?product=Nebula
> > > >
> > > > I'm assuming the code is under the Eclipse
> > Public License?
> > > >
> > > > When you file the bug, let us know.
> > > >
> > > > Cheers,
> > > >
> > > > --
> > > > Chris Aniszczyk | EclipseSource Austin |
> +1 860
> > 839 2465
> > > > http://twitter.com/eclipsesource |
> > > http://twitter.com/caniszczyk
> > > >
> > > >
> _______________________________________________
> > > > nebula-dev mailing list
> > > > nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>
> > <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>> <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>
> > > <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx> <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>>
> > > >
> https://dev.eclipse.org/mailman/listinfo/nebula-dev
> > > >
> > > >
> > > >
> > > > _______________________________________________
> > > > nebula-dev mailing list
> > > > nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>
> > <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>> <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>
> > > <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx> <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>>
> > > >
> https://dev.eclipse.org/mailman/listinfo/nebula-dev
> > > >
> > > >
> > > >
> > > > _______________________________________________
> > > > nebula-dev mailing list
> > > > nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx> <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>
> > > <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx> <mailto:nebula-dev@xxxxxxxxxxx
> <mailto:nebula-dev@xxxxxxxxxxx>>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>>
> > > > https://dev.eclipse.org/mailman/listinfo/nebula-dev
> > > >
> > > >
> > > >
> > > >
> > >
> >
> ------------------------------------------------------------------------
> > > >
> > > > _______________________________________________
> > > > nebula-dev mailing list
> > > > nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>
> > > > https://dev.eclipse.org/mailman/listinfo/nebula-dev
> > >
> > > _______________________________________________
> > > nebula-dev mailing list
> > > nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>
> > <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>>
> > > https://dev.eclipse.org/mailman/listinfo/nebula-dev
> > >
> > >
> > >
> > >
> >
> ------------------------------------------------------------------------
> > >
> > > _______________________________________________
> > > nebula-dev mailing list
> > > nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>
> > > https://dev.eclipse.org/mailman/listinfo/nebula-dev
> >
> > _______________________________________________
> > nebula-dev mailing list
> > nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> <mailto:nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>>
> > https://dev.eclipse.org/mailman/listinfo/nebula-dev
> >
> >
> >
> >
> ------------------------------------------------------------------------
> >
> > _______________________________________________
> > nebula-dev mailing list
> > nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> > https://dev.eclipse.org/mailman/listinfo/nebula-dev
>
> _______________________________________________
> nebula-dev mailing list
> nebula-dev@xxxxxxxxxxx <mailto:nebula-dev@xxxxxxxxxxx>
> https://dev.eclipse.org/mailman/listinfo/nebula-dev
>
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> nebula-dev mailing list
> nebula-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/nebula-dev