paches for Tags and Picture handling on the profile

Anton Lundin glance at acc.umu.se
Thu Jul 10 11:04:32 PDT 2014


On 10 July, 2014 - Tomaz Canabrava wrote:

> I still need to hoock the toggle on the image button to actually hide
> / show the pictures.

> From a461294b2d6117721fa4e83742821ecac945d78e Mon Sep 17 00:00:00 2001
> From: Tomaz Canabrava <tomaz.canabrava at intel.com>
> Date: Thu, 10 Jul 2014 13:50:49 -0300
> Subject: [PATCH 5/6] Added the toggle picture button and hoocked it up
> 
> This patch adds the toggle picture button and hoocks
> it up with the rest of the code. I'v also changed a call
> from ProfileWidget because it caused errors on the ui
> generated code, where it would try to call an still-to-be
> instantiated object.
> 
> Signed-off-by: Tomaz Canabrava <tomaz.canabrava at intel.com>
> ---
>  pref.h                           |  1 +
>  qt-ui/mainwindow.cpp             |  7 +++++++
>  qt-ui/mainwindow.h               |  2 +-
>  qt-ui/mainwindow.ui              | 31 +++++++++++++++++++++++++++----
>  qt-ui/profile/profilewidget2.cpp |  1 -
>  subsurfacestartup.c              |  3 ++-
>  6 files changed, 38 insertions(+), 7 deletions(-)
> 
> diff --git a/pref.h b/pref.h
> index a22206d..bfc120c 100644
> --- a/pref.h
> +++ b/pref.h
> @@ -63,6 +63,7 @@ struct preferences {
>  	char *proxy_pass;
>  	bool doo2breaks;
>  	bool drop_stone_mode;
> +	bool show_pictures_in_planner;

Why name it _in_planner?

As far as i understood it got nothing with the planner to do. Maybe call
it show_pictures_in_profile?

//Anton


-- 
Anton Lundin	+46702-161604


More information about the subsurface mailing list