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