[Date Prev][Date Next][Thread Prev][Thread Next][Thread Index]
RE: [XaraXtreme-dev] RE: [XaraXtreme-bugzilla] [Bug 1125] Bevel tool does not work properly on Text
- From: "Phil Martin" <Phil@xxxxxxxx>
- Date: Tue, 4 Jul 2006 21:19:45 +0100
- Subject: RE: [XaraXtreme-dev] RE: [XaraXtreme-bugzilla] [Bug 1125] Bevel tool does not work properly on Text
I encountered this when implementing LiveEffects in Xtreme and had to be
very careful about the use of IsCompound and I created IsCompoundClass
at this time to detect real compound nodes...
I think BaseTextClass used to return TRUE from IsCompound and thus so
did TextChar. If I remember correctly I changed that in Xtreme at the
time of implementing LiveEffects.
So if it still exists in LX it's a throwback/lost change.
FYI: in Xtreme now only NodeCompound, TextLine and TextStory return TRUE
from IsCompound.
Phil
> -----Original Message-----
> From: owner-dev@xxxxxxxxxxxxxxxx
> [mailto:owner-dev@xxxxxxxxxxxxxxxx] On Behalf Of Gerry Iles
> Sent: 04 July 2006 20:35
> To: dev@xxxxxxxxxxxxxx
> Subject: RE: [XaraXtreme-dev] RE: [XaraXtreme-bugzilla] [Bug
> 1125] Bevel tool does not work properly on Text
>
> Well, it might have been but the change makes no sense
> conceptually. A TextChar object is not a compound node and
> never can be so any change that makes it so is fundamentally wrong...
>
> I think it was either accidental (e.g. the function was
> accidentally moved from the TextStory class) or not very well
> considered due to unfamiliarity with the code...
>
> Gerry
>
> -----Original Message-----
> From: owner-dev@xxxxxxxxxxxxxxxx
> [mailto:owner-dev@xxxxxxxxxxxxxxxx] On Behalf Of Alex Bligh
> Sent: 04 July 2006 18:49
> To: dev@xxxxxxxxxxxxxx
> Cc: Alex Bligh
> Subject: Re: [XaraXtreme-dev] RE: [XaraXtreme-bugzilla] [Bug
> 1125] Bevel tool does not work properly on Text
>
> Gerry Iles wrote:
> > Martin,
> >
> > In fact, it appears that this override of IsCompound was present in
> r714
> > which was before any of your text handling stuff was added.
> >
> > Anyone remember the error "LocaliseCommonAttributes called on
> > non-Compound Node" when doing things with text stories
> early on in the
> > port?
>
> I have better archaeology. It was thus on rev 385 of the very
> first svn repository. It must have been a very early change
> indeed. Which is most peculiar as r385 was nowhere near
> rendering text.
>
> Are you sure it was not at some stage converted to do that in
> SourceSafe, someone took the patch over, then SourceSafe got
> reverted but no-one carried the patch through to svn?
>
> Alex
>