On Wed, Jul 8, 2015 at 11:59 AM, Dirk Hohndel <[email protected]> wrote:

> On Wed, Jul 08, 2015 at 11:20:43AM -0300, Tomaz Canabrava wrote:
> > this doesn't selects the correct dive site yet if you have more than one
> > dive site with the same name, it's just a proof of concept about the
> visual
> > of it.
>
> I LOVE the visual. This is really strong, Tomaz. Thanks for your work on
> this.
>
> > I'm working on make it select the right dive_site.
>
> I notice one oddity. I have one dive site name three times with three
> slightly different GPS coordinates. You do show it three times as
> completion option, but you show it with the same GPS coordinates each
> time...
>
> I haven't spent the time to look at the code - my bet is that this is
> something trivial and easy to fix.
>
> /D
>
From 9671140daddcaba38345e2c8afb038ae011b81c6 Mon Sep 17 00:00:00 2001
From: Tomaz Canabrava <[email protected]>
Date: Wed, 8 Jul 2015 11:59:46 -0300
Subject: [PATCH 5/5] use current dive_site to get the information

and not the displayed_dive_site. tsc.

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

diff --git a/qt-ui/modeldelegates.cpp b/qt-ui/modeldelegates.cpp
index 2c03a80..2e9b6d6 100644
--- a/qt-ui/modeldelegates.cpp
+++ b/qt-ui/modeldelegates.cpp
@@ -512,7 +512,7 @@ void LocationFilterDelegate::paint(QPainter *painter, const QStyleOptionViewItem
 	}
 
 	if (bottomText.isEmpty()) {
-		const char *gpsCoords = printGPSCoords(displayed_dive_site.latitude.udeg, displayed_dive_site.longitude.udeg);
+		const char *gpsCoords = printGPSCoords(ds.latitude.udeg, ds.longitude.udeg);
 		bottomText = QString(gpsCoords);
 		free( (void*) gpsCoords);
 	}
-- 
2.4.5

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

Reply via email to