[PATCH rhel7-branch] (take 2) gui: add Refresh button to network storage

Radek Vykydal rvykydal at redhat.com
Tue Sep 16 07:25:12 UTC 2014


On 09/16/2014 08:56 AM, Vratislav Podzimek wrote:
> On Mon, 2014-09-15 at 13:58 +0200, Radek Vykydal wrote:
>> Following review of first take on the patch I ported timed_action
>> patches from master. 3/4/ and 4/4 are optional, I was thinking let's
>> port them too when porting the feature.
> Agreed. I think it improves UX and shouldn't cause any harm. But didn't
> you forget to use the decorator for the button-press handler which was
> the original reason for adding it?
>

It's in 2/4 but I should have ported timed_action first and
then use timed_action in the patch for Refresh button,
I'm going to reorganize the patches this way for clearer
history.


More information about the anaconda-patches mailing list