[PATCH 1/4] Profile2: provide means to disable the tool buttons

Dirk Hohndel dirk at hohndel.org
Tue Mar 11 11:30:34 PDT 2014


On Tue, 2014-03-11 at 15:26 -0300, Tomaz Canabrava wrote:

>         >
>         > Once the poster is displayed when there are no dives in the
>         list,
>         > we may also want to disable the QToolButtons (PO2, SAC,
>         etc..),
>         > until a new dive is loaded and the profile is redrawn.
>         
>         
>         Wouldn't it be easier to simply hide / show the vertical
>         layout the
>         buttons are in?
>         
>         This seems really a bit awkward...
> 
> 
> Not as much - if you hide there's a form-factor change of the profile,
> things can 'jump' a bit, this way the unused / not enabled buttons
> will be gray untill they are set enabled.
>  
Ok, that's a good argument

I'll take the patch as is...

/D




More information about the subsurface mailing list