[api-changes] [Bug 113456] Add support for rotating Widgets

  • From:
  • To:
  • Subject: [api-changes] [Bug 113456] Add support for rotating Widgets
  • Date: Sun, 25 Nov 2012 20:36:44 +0000
  • Auto-submitted: auto-generated


--- Comment #24 from bcallebaut 
< >
 2012-11-25 20:36:44 UTC ---
(In reply to comment #22)
> > * I didn't want to add another constructor in the Widget base class. The
> > constructor using a Lookup is already present, so it is easy to reuse it 
> > to
> > pass an unknown set of instances to the Widget. I agree this is not API
> > tunneling
> Why not? It seems to be better if the constructor signature describes what
> objects are expected.
> Passing border and layout is probably beneficial, because instantiation of
> possibly unused EmptyBorder and AbsoluteLayout can be skipped, and Lookup 
> can
> be useful for putting custom objects ("cookies") to the widget.
> But I'm not sure that passing ChainProviders to the constructor is really
> needed. Isn't using createActions() and getActions().addAction() sufficient?

> > * I wanted to have a way to "configure" a widget (standard 
> > border,actions....)
> > in an easy way without having to add after construction many lines of 
> > code. In
> > that  scope I use the Lookup object as a kind of Map<Class,Object> with 
> > more
> > than 1 value per key that can be shared by multiple widget classes.
> You can create a helper method that sets the widget as you wish. It may be
> simpler and faster than creating a Lookup and then reading it during
> construction of each object.

> Something like this should be safe:
For me it is good. I just don't want to break existing compiled code.

> > * I have use case where the Lookup object is tunneled (WidgetFactory).
> You can have a method (in your widget factory) that initializes newly 
> created
> widget, e.g. sets actions. Is this solution not suitable for you?

A widget factory is in my plan. I don't know yet how to fit it in the general
API. I still have to think it and try.

Configure bugmail: http://netbeans.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[api-changes] [Bug 113456] Add support for rotating Widgets

bcallebaut 11/25/2012

Project Features

About this Project

www was started in November 2009, is owned by jpirek, and has 25 members.
By use of this website, you agree to the NetBeans Policies and Terms of Use (revision 20150226.965aeb8). © 2014, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo
Please Confirm