StatusBar in KMainWindow
Simon Hausmann
hausmann at kde.org
Thu Mar 13 17:47:19 GMT 2003
On Thu, Mar 13, 2003 at 12:03:21PM -0500, Benjamin Meyer wrote:
> On Thursday 13 March 2003 11:28 am, Simon Hausmann wrote:
> > On Thu, Mar 13, 2003 at 10:00:52AM -0500, Benjamin Meyer wrote:
> > > Attached is the new diff which has the new function that is used to
> > > create the Action.
> >
> > I like that approach, for its consistency with the toolbar handling.
> >
> > Three minor things though:
> >
> > 1) The set method should take a bool and it should also handle the
> > case where it's called with false IMHO. Hmm, not terribly easy
> > though, maybe this should be done simliar to the toolbar
> > handling, except for using an actionlist instead of a whole
> > guiclient component.
>
> Can I simply call delete on the KAction? Will everything be ok?
Nope, that would make the xmlgui engine end up with a dangling
pointer and leave the action plugged in the GUI.
> > 3) The documentation (apart from that it talks about a menu)
> > mentions the requirement to connect to a signal of the action.
> > Looking at the patch it seems this is done automatically already
> > (apart from that there is no way for the developer to retrieve
> > a reference to the action because there is no accessor) .
>
> That last paragraph is for those apps/classes who *don't* use the function,
> but handle the KAction themselves. In that case they needs to make the
> connection. Is there a better way I could word it to make that more clear?
Ah, sorry, misunderstood :)
(thanks also Joseph for hinting)
Simon
More information about the kde-core-devel
mailing list