[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: KWord Doc Structure Area refreshing
From:       Thomas Zander <zander () kde ! org>
Date:       2005-12-24 8:13:13
Message-ID: 200512240913.17769.zander () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 23 December 2005 22:19, Gary Cramblitt wrote:
> The simplest way to get the dsa to refresh was to emit a signal
> whenever a command is added to the command history, hence I added
> commandHistoryChanged signal to kocommandhistory.

That kind of looks like a nice trick, I'm not sure that it makes sense 
design wise :)  (i.e. does not break in unpredictable ways in the future)

I'd like to point out the signals from KWFrameViewManager[1] (which is a 
member of KWCanvas and also reachable in KWView[2]).
In the frameViewManager there are more slots then there are signals; thats 
because the class is currently in flux; the KWFramesListener[3] can be 
inherited to get the rest of the signals.  Feel free to convert those 
callback methods to signals in the viewManager, if this concept is weird 
to you. (see the fireEvents method for that)

Another issue;
From a design perspective, its the responsibility of the class that does 
the new KWDocStruct() to do the connects, adding a connect in the 
constructor of the widget looks ugly and out of place to me.

Hope these hints help;


1) 
http://www.koffice.org/developer/apidocs/kword/html/classKWFrameViewManager.html
 but also (more uptodate) on;
http://www.englishbreakfastnetwork.org/apidocs/apidox-koffice/kword-apidocs/classKWFrameViewManager.html


2) 
http://www.koffice.org/developer/apidocs/kword/html/classKWView.html#a64

3) 
http://www.koffice.org/developer/apidocs/kword/html/classKWFramesListener.html \
                
-- 
Thomas Zander


[Attachment #5 (application/pgp-signature)]

_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel


[prev in list] [next in list] [prev in thread] [next in thread]