Re: CVS: commit robsta abi/src/text/fmt/xp fv_View.h fv_VisualDragText.h

On 1/23/07, Tomas Frydrych <tf@o-hand.com> wrote:
>
> Hi Rob,
>
> > class FL_DocLayout;
> > @@ -978,7 +983,11 @@
> > fv_PropCache m_SecProps;
> > AV_ListenerId m_CaretListID;
> > FV_FrameEdit m_FrameEdit;
> > +#ifdef XP_UNIX_TARGET_GTK
> > + FV_UnixVisualDrag m_VisualDragText;
> > +#else
> > FV_VisualDragText m_VisualDragText;
> > +#endif
>
> Why is this necessary? I assume there are good reasons for this, but I
> am very uneasy about introducing platform specific code into src/text,
> and particularly src/text/fmt, so I would like to know bit more about
> these changes. Thanks.

This changes have been made to let platforms take advantage of native dnd APIs.
Whether src/text/fmt is the right place for dnd in the first place is
debatable, of course. I've added it there because it already was
there.

Best,
Rob
Received on Tue Jan 23 21:44:37 2007