correct ones: I'm doing the rest of the code now ( to make the
pictures actually appear / disappear on the canvas )

On Thu, Jul 10, 2014 at 6:33 PM, Tomaz Canabrava <[email protected]> wrote:
> my mistake. :)
>
> On Thu, Jul 10, 2014 at 3:04 PM, Anton Lundin <[email protected]> wrote:
>> 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 <[email protected]>
>>> 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 <[email protected]>
>>> ---
>>>  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
From a461294b2d6117721fa4e83742821ecac945d78e Mon Sep 17 00:00:00 2001
From: Tomaz Canabrava <[email protected]>
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 <[email protected]>
---
 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;
 };
 enum unit_system_values {
 	METRIC,
diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp
index 6a212ae..bb65ada 100644
--- a/qt-ui/mainwindow.cpp
+++ b/qt-ui/mainwindow.cpp
@@ -93,6 +93,7 @@ MainWindow::MainWindow() : QMainWindow(),
 #endif
 
 	ui.mainErrorMessage->hide();
+	ui.newProfile->setEmptyState();
 	initialUiSetup();
 	readSettings();
 	ui.ListWidget->reload(DiveTripModel::TREE);
@@ -1334,6 +1335,12 @@ void MainWindow::on_profScaled_clicked(bool triggered)
 	TOOLBOX_PREF_PROFILE(zoomed_plot);
 }
 
+void MainWindow::on_profTogglePicture_clicked(bool triggered)
+{
+	prefs.show_pictures_in_planner = triggered;
+	TOOLBOX_PREF_PROFILE(show_pictures_in_planner);
+}
+
 #undef TOOLBOX_PREF_PROFILE
 
 void MainWindow::on_actionExport_triggered()
diff --git a/qt-ui/mainwindow.h b/qt-ui/mainwindow.h
index 014fbb4..3e7cad6 100644
--- a/qt-ui/mainwindow.h
+++ b/qt-ui/mainwindow.h
@@ -142,7 +142,7 @@ slots:
 	void on_profRuler_clicked(bool triggered);
 	void on_profSAC_clicked(bool triggered);
 	void on_profScaled_clicked(bool triggered);
-
+	void on_profTogglePicture_clicked(bool triggered);
 	void on_actionExport_triggered();
 
 protected:
diff --git a/qt-ui/mainwindow.ui b/qt-ui/mainwindow.ui
index 2360dd8..d8ad99c 100644
--- a/qt-ui/mainwindow.ui
+++ b/qt-ui/mainwindow.ui
@@ -109,7 +109,7 @@
          <property name="spacing">
           <number>0</number>
          </property>
-         <item row="14" column="0">
+         <item row="15" column="0">
           <spacer name="verticalSpacer">
            <property name="orientation">
             <enum>Qt::Vertical</enum>
@@ -466,6 +466,9 @@
            </property>
           </widget>
          </item>
+         <item row="0" column="1" rowspan="16">
+          <widget class="ProfileWidget2" name="newProfile"/>
+         </item>
          <item row="1" column="0">
           <widget class="QToolButton" name="profPn2">
            <property name="toolTip">
@@ -492,8 +495,28 @@
            </property>
           </widget>
          </item>
-         <item row="0" column="1" rowspan="15">
-          <widget class="ProfileWidget2" name="newProfile"/>
+         <item row="14" column="0">
+          <widget class="QToolButton" name="profTogglePicture">
+           <property name="text">
+            <string>...</string>
+           </property>
+           <property name="icon">
+            <iconset resource="../subsurface.qrc">
+             <normaloff>:/pictures</normaloff>:/pictures</iconset>
+           </property>
+           <property name="iconSize">
+            <size>
+             <width>24</width>
+             <height>24</height>
+            </size>
+           </property>
+           <property name="checkable">
+            <bool>true</bool>
+           </property>
+           <property name="autoRaise">
+            <bool>true</bool>
+           </property>
+          </widget>
          </item>
         </layout>
        </widget>
@@ -638,7 +661,7 @@ p, li { white-space: pre-wrap; }
      <x>0</x>
      <y>0</y>
      <width>1682</width>
-     <height>19</height>
+     <height>27</height>
     </rect>
    </property>
    <widget class="QMenu" name="menuFile">
diff --git a/qt-ui/profile/profilewidget2.cpp b/qt-ui/profile/profilewidget2.cpp
index ee80d59..fe19c7b 100644
--- a/qt-ui/profile/profilewidget2.cpp
+++ b/qt-ui/profile/profilewidget2.cpp
@@ -100,7 +100,6 @@ ProfileWidget2::ProfileWidget2(QWidget *parent) : QGraphicsView(parent),
 	setupItemOnScene();
 	addItemsToScene();
 	scene()->installEventFilter(this);
-	setEmptyState();
 	connect(PreferencesDialog::instance(), SIGNAL(settingsChanged()), this, SLOT(settingsChanged()));
 
 	QAction *action = NULL;
diff --git a/subsurfacestartup.c b/subsurfacestartup.c
index 58291d2..df4c01f 100644
--- a/subsurfacestartup.c
+++ b/subsurfacestartup.c
@@ -41,7 +41,8 @@ struct preferences default_prefs = {
 	.bottompo2 = 1400,
 	.decopo2 = 1600,
 	.doo2breaks = false,
-	.drop_stone_mode = false
+	.drop_stone_mode = false,
+	.show_pictures_in_planner = true
 };
 
 int run_survey;
-- 
2.0.1

From c5a60d533f3fad12426215f270222a7ecb3c7f21 Mon Sep 17 00:00:00 2001
From: Tomaz Canabrava <[email protected]>
Date: Thu, 10 Jul 2014 13:53:06 -0300
Subject: [PATCH 6/6] Correctly sets the 'enabled' flag on the picture button

Forgot to add this one, oops.

Signed-off-by: Tomaz Canabrava <[email protected]>
---
 qt-ui/mainwindow.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp
index bb65ada..8c86901 100644
--- a/qt-ui/mainwindow.cpp
+++ b/qt-ui/mainwindow.cpp
@@ -241,6 +241,7 @@ void MainWindow::setToolButtonsEnabled(bool enabled)
 	ui.profRuler->setEnabled(enabled);
 	ui.profScaled->setEnabled(enabled);
 	ui.profHR->setEnabled(enabled);
+	ui.profTogglePicture->setEnabled(enabled);
 }
 
 bool MainWindow::okToClose(QString message)
-- 
2.0.1

_______________________________________________
subsurface mailing list
[email protected]
http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to