StatusBar in KMainWindow
Benjamin Meyer
ben at meyerhome.net
Thu Mar 13 17:03:21 GMT 2003
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?
> 2) I'm not sure about the name, because in despite of what the name
> suggests this isn't about a whole menu but a single action (a single
> menu item or toolbar button usually) . How about
> setStandardStatusBarActionEnabled?
Changed.
> 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?
>
> Simon
More information about the kde-core-devel
mailing list