in-place-editing patch for konqueror bookmarks

[David Faure]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Friday 17 January 2003 18:38, Oelewapperke wrote:
> On Friday 17 January 2003 11:02, David Faure wrote:
> > On Thursday 16 January 2003 23:02, Oelewapperke wrote:
> > > On Thursday 16 January 2003 18:07, David Faure wrote:
> > > > Please reconsider this. You rewrote a complete bookmarkbar
> > > > implementation just to change the menus! There is much duplicated code
> > > > in your patch, which I'm strongly against. This will become a
> > > > maintainance hell.
> > >
> > > This is not true, check the code, I changed both the buttons and the
> > > menus (and as far as I can see I need to do that)
> >
> > Ok, the KIPEToolBarButton indeed adds MMB/RMB to the buttons.
> > But why a brand new KIPEBookmarkBar? Why not integrate this with
> > the original bookmarkbar? We don't need to have two of them...
> 
> because the class delivers events, something which can not be done with only a
> KBookmarkOwner, I need a KIPEBookmarkOwner for that.

Implement it with signals, it'll be much simpler (since it will introduce 
no BIC problem, it can be done in the existing classes)

- -- 
David Faure -- faure at kde.org, dfaure at klaralvdalens-datakonsult.se
Klarälvdalens Datakonsult AB, Platform-independent software solutions
Contributing to: http://www.konqueror.org/, http://www.koffice.org/
KOffice-1.2.1 is available - http://download.kde.org/stable/koffice-1.2.1/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE+KE8Q72KcVAmwbhARAuSGAJ4zPfr8y9ZOx337ra81tYRMwr3wigCeJIri
DyYgKXswe9RnfCsV7mroGaw=
=hM1e
-----END PGP SIGNATURE-----