On 09/16/2014 01:18 PM, Vratislav Podzimek wrote:
On Tue, 2014-09-16 at 11:48 -0400, Chris Lumens wrote:
This doesn't take into account the possibility that the user wants to redo custom storage without changing any of the settings on the main storage spoke. Otherwise I like this one.
And in fact, there is a comment to this effect in storage.py:
def on_back_clicked(self, button): # We can't exit early if it looks like nothing has changed because the # user might want to change settings presented in the dialogs shown from # within this method.
The only use case that comes to my mind is if user wants to reclaim some more space, but okay, dropping this patch.
Maybe you can just add 'or self._reclaim.get_active()' to the same if statement you added 'or self._customPart.get_active()' to instead of dropping the patch completely.
David