From kde-core-devel  Sat Oct 21 04:38:39 2006
From: Hamish Rodda <rodda () kde ! org>
Date: Sat, 21 Oct 2006 04:38:39 +0000
To: kde-core-devel
Subject: Re: A KSelectAction dedicated to QTextCodec selection ?
Message-Id: <200610211438.46288.rodda () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=116140559229273
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--nextPart2268659.xWKqbN64J6"

--nextPart2268659.xWKqbN64J6
Content-Type: text/plain;
  charset="iso-8859-6"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

On Saturday 21 October 2006 05:36, Michel Hermier wrote:
> Hi,
>
> 2006/10/20, David Faure <faure@kde.org>:
> > Hello!
> >
> > On Friday, October 20, 2006 01:19:05 PM Michel Hermier wrote:
> > > I noticed while preparing the changes locally, that I need one of my
> > > local change for KSelectAction (well only a return change, but the
> > > patch adds a little more).
> > > So I delay the commit for now.
> > >
> > > Can someone maintaining KSelectAction review the patch.
> > > What changed with this patch:
> > > - bool setCurrentAction(QAction *, DeselectionMode mode) should be
> > > more safe, checking that the action really belongs to the action group
> > > before activating the action. Also added an extra parameter with
> > > default value to mimic the old behaviour. This extra parameter allow
> > > to not deselect the previous action in case of falure to select the
> > > action.
> >
> > Why does this need to be configurable? Deselection the previous action
> > and not selecting any new one instead looks like a bug, not a feature.

No, unselecting the current action is needed in the case where we want to=20
clear the current selection.  For example, in KRecentFilesAction, when a fi=
le=20
is closed, its action is deselected so it is no longer checked.

> > Did you make it configurable "just in case", or is there a real use case
> > for this?
>
> I think it's needed to be configurable if we allow to have multiple
> selection enabled.
> Because in this case you have to look at all the actions to search to
> know wich ones are enabled, since the API don't offer to get all the
> selected items (directly).

KSelectAction is not designed to allow multiple selection.

> And yes there is a real need to allow multiple selection, since
> usually these groups are quite large (see the font and QTextCodec). So
> we need an option to configure the removal some of them, so we need to
> select multiple items.

Either have a separate way to configure the action (eg. configuration dialo=
g),=20
or create a new class which does what you want.

As to the earlier patch, I think the part which checks that the new current=
=20
action is one of the selectable actions is worthwhile.

Cheers,
Hamish.

--nextPart2268659.xWKqbN64J6
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)

iD8DBQBFOaRWH8BtnSmIlUYRAmTCAKCYOtWRg1GH+HcJGDlWyqvGvt58WQCg33vL
CyD8muH7paaQv/pbf67z6+4=
=gxji
-----END PGP SIGNATURE-----

--nextPart2268659.xWKqbN64J6--