[rhel7-branch][PATCH] Keep selection when switching environments
Vratislav Podzimek
vpodzime at redhat.com
Fri Mar 7 08:27:41 UTC 2014
On Thu, 2014-03-06 at 17:28 +0100, Martin Kolman wrote:
> As master has addon switching handled differently due to newer GTK,
> this is rhel7-branch (and most probably 7.1 due to the GUI changes) only.
>
> The software spoke current behavior is this:
> * every time you switch to another environment, you selection is
> erased and replaced by defaults for the given spoke (if the
> current comps and environment that has any addons set as default)
> * even if you previously selected an addon that is also available
> in the new environment, you selection will still be cleared
> * defaults for the given environment will be always selected again
> when switching, even if you deselected them previously
>
> The patch fixes these issues so that:
> * once you select an addon, it will stay selected in all environments
> you switch into that include the given addon
> * the same thing for deselecting defaults - once you deselect a default,
> it will be deselected in all environments that have it enabled by default
> * of course once you click on "done" only the addons in the currently ticked
> environment will be used for the install
>
> Also regarding the implementation, I've replaced the two lists (one of them
> was for some reason not being used for anything) by dict and decoupled
> user selection from getting list of currently selected addons in apply,
> making it possible to remove quite a few lines of code. :)
>
> Martin Kolman (1):
> Keep selection when switching environments
>
> pyanaconda/ui/gui/spokes/software.py | 43 +++++++++++++-----------------------
> 1 file changed, 15 insertions(+), 28 deletions(-)
Other than those few rather neatpicks this looks like a very nice change. ACK!
--
Vratislav Podzimek
Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic
More information about the anaconda-patches
mailing list