kfinddialog patch

[Frans Englich]

On Thursday 09 September 2004 20:36, David Faure wrote:
> On Thursday 09 September 2004 22:37, Frans Englich wrote:
> > On Thursday 09 September 2004 20:25, David Faure wrote:
> > > On Thursday 09 September 2004 22:19, Albert Astals Cid wrote:
> > > > Something like that?
> > > >
> > > > Please review again
> > >
> > > Looks good, thanks.
> >
> > Would it be a good idea to also make the public API use enums, instead of
> > a number of bool functions?
> >
> > For example, enabling a feature would be done like this:
> >
> > setCapabilities( KFindDialog::BackwardsFind | capabilities() );
>
> That works for app-capabilities, but when it comes to e.g. "I have a
> selection", "now I don't have a selection"... those settings change over
> time, so it's easier with a bool setter IMHO.

Yes, good point. They are more dynamic and separated from each other.


			Frans