[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: Improving ktexteditor interface (patch included)
From: "Jos Poortvliet" <jos () mijnkamer ! nl>
Date: 2007-08-27 11:03:58
Message-ID: 5c77e14b0708270403r3cf1e4bbqe2f0af8be34242ce () mail ! gmail ! com
[Download RAW message or body]
On 8/23/07, Philippe Fremy <phil@freehackers.org> wrote:
>
>
> Hi,
>
> It's been 5 days and nobody answered. I interpret this as "nobody really
> has an opinion about this to really comment on it".
Seems like you still got it, then - everyone still likes your work, and
nobody complains ;-)
The patch was discussed and approved on ktexteditor, so, if there is no
> comment from kde-core-devel, I'll commit it on next monday.
>
> That would be my first patch to KDE since 1999 :-)
>
> Philippe
>
>
> Philippe Fremy wrote:
> > Hi,
> >
> > I propose the following patch to improve the ktexteditor interface.
> >
> > The current ktexteditor interface is based on a master -> slave
> > relationship. The kpart host decides what to do (open documents, open
> > views, close them, ...) and the kpart component obeys.
> >
> > When working with Yzis (a vi engine for text editing, see
> > http://www.yzis.org ), this does not work very well because Yzis is not
> > a good slave. It wants to do stuff on his own:
> > - with :edit some_file.txt , the kpart wants to open a new document and
> > a new view
> > - with :bdelete, the kpart wants to remove a document and a view
> > - with :bnext, the kpart wants to change the active view
> >
> > The patch allows the kpart host to set a class instance, which can be
> > used for kpart component to kpart host communication. The mechanism is
> > based on qobject_cast - like other extension mechanism in ktexteditor -
> > and allows for current and future extensions of the kpart host interface
> > in a BC way.
> >
> > The patch has been discussed on ktexteditor (hence the seventh
> > iteration) and has the approval of the people there ( see
> > http://lists.kde.org/?l=ktexteditor-devel&m=118724435408905&w=2 ).
> >
> > The first extension supported is the MdiContainer, that allows the part
> > to create views, delete views, create documents, delete documents,
> > change and fetch the current active view. The extension mechanism is by
> > definition completely optional for the kpart host.
> >
> > The patch is BC. Eventhough we have an API freeze, I would like it to go
> > into KDE 4.0 .
> >
> > cheers,
> >
> > Philippe
> >
> >
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > KTextEditor-Devel mailing list
> > KTextEditor-Devel@kde.org
> > https://mail.kde.org/mailman/listinfo/ktexteditor-devel
>
>
>
[Attachment #3 (text/html)]
<br><br><div><span class="gmail_quote">On 8/23/07, <b \
class="gmail_sendername">Philippe Fremy</b> <<a \
href="mailto:phil@freehackers.org">phil@freehackers.org</a>> \
wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, \
204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> \
<br> Hi,<br><br>It's been 5 days \
and nobody answered. I interpret this as "nobody really<br>has an opinion about \
this to really comment on it".</blockquote><div><br>Seems like you still got it, \
then - everyone still likes your work, and nobody complains ;-) \
<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, \
204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">The patch was discussed and \
approved on ktexteditor, so, if there is no<br>comment from kde-core-devel, I'll \
commit it on next monday. <br><br>That would be my first patch to KDE since 1999 \
:-)<br><br> Philippe<br><br><br>Philippe \
Fremy wrote:<br>> Hi,<br>><br>> I \
propose the following patch to improve the ktexteditor interface.<br> ><br>> \
The current ktexteditor interface is based on a master -> slave<br>> \
relationship. The kpart host decides what to do (open documents, open<br>> views, \
close them, ...) and the kpart component obeys.<br> ><br>> When working with \
Yzis (a vi engine for text editing, see<br>> <a \
href="http://www.yzis.org">http://www.yzis.org</a> ), this does not work very well \
because Yzis is not<br>> a good slave. It wants to do stuff on his own: <br>> - \
with :edit some_file.txt , the kpart wants to open a new document and<br>> a new \
view<br>> - with :bdelete, the kpart wants to remove a document and a view<br>> \
- with :bnext, the kpart wants to change the active view <br>><br>> The patch \
allows the kpart host to set a class instance, which can be<br>> used for kpart \
component to kpart host communication. The mechanism is<br>> based on qobject_cast \
- like other extension mechanism in ktexteditor - <br>> and allows for current and \
future extensions of the kpart host interface<br>> in a BC way.<br>><br>> \
The patch has been discussed on ktexteditor (hence the seventh<br>> iteration) and \
has the approval of the people there ( see <br>> <a \
href="http://lists.kde.org/?l=ktexteditor-devel&m=118724435408905&w=2">http://lists.kde.org/?l=ktexteditor-devel&m=118724435408905&w=2</a> \
).<br>><br>> The first extension supported is the MdiContainer, that allows the \
part <br>> to create views, delete views, create documents, delete \
documents,<br>> change and fetch the current active view. The extension mechanism \
is by<br>> definition completely optional for the kpart host.<br>> <br>> The \
patch is BC. Eventhough we have an API freeze, I would like it to go<br>> into KDE \
4.0 .<br>><br>> \
cheers,<br>><br>> \
Philippe<br>><br>><br>><br>> \
------------------------------------------------------------------------ \
<br>><br>> _______________________________________________<br>> \
KTextEditor-Devel mailing list<br>> <a \
href="mailto:KTextEditor-Devel@kde.org">KTextEditor-Devel@kde.org</a><br>> <a \
href="https://mail.kde.org/mailman/listinfo/ktexteditor-devel"> \
https://mail.kde.org/mailman/listinfo/ktexteditor-devel</a><br><br><br></blockquote></div><br>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic