| GraphicalEditorWithFlyoutPalette [message #151500] |
Mon, 20 September 2004 15:36  |
Eclipse User |
|
|
|
Originally posted by: james.willans.xactium.com
My palette is currently implemented using GraphicalEditorWithPalette,
however I would like to upgrade this to use
GraphicalEditorWithFlyoutPalette. I thought this would be a matter of
extending a different class and implementing the abstract method
getPalettePreferences(), however I've run into difficulties. I get a null
pointer exception in createPaletteViewerProvider() because
getGraphicalViewer() returns null at that point. Should I be doing
something that I'm not?
Thanks.
James
|
|
|
|
|
|
|
|
| Re: GraphicalEditorWithFlyoutPalette [message #151602 is a reply to message #151594] |
Tue, 21 September 2004 12:22  |
| Eclipse User |
|
|
|
Not unless you plan to copy the flyout-related classes. I'd recommend
against it, of course. We want all different GEF palettes to look the same
for the end-user.
"James Willans" <james.willans@xactium.com> wrote in message
news:cipj8d$33i$1@eclipse.org...
> Pratik,
>
> Thanks I have now got it working. Just a quick question, I would like to
> get rid of the palette's title "Palette" and just have the two lines
> shown. However when I change the externalised string to "" it looks odd
> because a space is left between the two halfs of the line. Is there any
> way of having unbroken lines when the string is ""?
>
> James
>
> Pratik Shah wrote:
>
> > That was fixed a while back to: return new
> > PaletteViewerProvider(getEditDomain());
> > You can override the method and make this change yourself, or you can
move
> > to the latest GEF integration build.
>
> > "James Willans" <james.willans@xactium.com> wrote in message
> > news:ciooml$94n$1@eclipse.org...
> > > Pratik,
> > >
> > > I don't require it, but the body of createPaletteViewerProvider() in
the
> > > implementation of GraphicalEditorWithFlyoutPalette requires it:
> > >
> > > protected PaletteViewerProvider createPaletteViewerProvider() {
> > > return new
PaletteViewerProvider(getGraphicalViewer().getEditDomain());
> > > }
> > >
> > > ?
> > >
> > > Thanks.
> > >
> > > James
> > >
> > > Pratik Shah wrote:
> > >
> > > > That's right, the PaletteViewerProvider is created before the
graphical
> > > > viewer (look at
> > > > GraphicalEditorWithFlyoutPalette#createPartControl(Composite )). Why
do
> > you
> > > > need the graphical viewer for the PaletteViewerProvider?
> > >
> > >
> > > > "James Willans" <james.willans@xactium.com> wrote in message
> > > > news:cinbfg$6n0$1@eclipse.org...
> > > > > My palette is currently implemented using
GraphicalEditorWithPalette,
> > > > > however I would like to upgrade this to use
> > > > > GraphicalEditorWithFlyoutPalette. I thought this would be a
matter of
> > > > > extending a different class and implementing the abstract method
> > > > > getPalettePreferences(), however I've run into difficulties. I
get a
> > null
> > > > > pointer exception in createPaletteViewerProvider() because
> > > > > getGraphicalViewer() returns null at that point. Should I be
doing
> > > > > something that I'm not?
> > > > >
> > > > > Thanks.
> > > > >
> > > > > James
> > > > >
> > >
> > >
>
>
|
|
|
|
|
| Re: GraphicalEditorWithFlyoutPalette [message #151647 is a reply to message #151640] |
Tue, 21 September 2004 18:52 |
| Eclipse User |
|
|
|
Originally posted by: james.willans.xactium.com
Bug: 74493
James Willans wrote:
> Will do. Just for clarification - You can change the word "Palette" since
> the string is externalised.
> James
> Randy Hudson wrote:
> > So James, open a bugzilla requesting that ALL palettes not display
"Palette"
> > title when expanded. We'll check with our UI experts for a ruling on this.
> > "Pratik Shah" <ppshah@us.ibm.com> wrote in message
> > news:cipk82$5eq$1@eclipse.org...
> > > Not unless you plan to copy the flyout-related classes. I'd recommend
> > > against it, of course. We want all different GEF palettes to look the
> > same
> > > for the end-user.
> > >
> > >
|
|
|
Powered by
FUDForum. Page generated in 0.03233 seconds
Home