<div dir="ltr">Hi,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On 18 July 2015 at 06:47, Salvador Cuñat <span dir="ltr"><<a href="mailto:salvador.cunat@gmail.com" target="_blank">salvador.cunat@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Good night.<br>
<span class=""><br>
2015-07-17 19:38 GMT+02:00, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>>:<br>
> On Fri, Jul 17, 2015 at 10:27:19AM -0700, Dirk Hohndel wrote:<br>
><br>
</span><span class="">> Having said all this, yes, the patch fixes the bug. So to those of you<br>
> considering testing this (you ARE going to test this, right? after all the<br>
> fighting and yelling...) - please pull the latest master or get the -1141<br>
> daily builds that are being created as I speak.<br>
><br></span></blockquote><div><br></div><div>I just tested with the lastest master.<br><br>Tomaz and Dirk, thank you very much for implementing this.  A few days ago I doubted it would be possible to develop a solution that everybody would keep everybody happy.  I think it is great.  Hopefully there'll be feedback from both sides of the "duplicate site names are stupid" vs "coordinates for different parts of a site are important to me" debate.<br><br></div><div>As all my dives already have locations, I tested by copying my file, deleting my 15 most recent dives, then re-downloading them from my dive computer.  At first I was suprised that there was no option to autocomplete the name without assigning the existing site.  But on re-assigning a site to a dive (so the dive currently already has a location), that option appears.  I assume that's what will happen if the companion app is used and the dive has a GPS fix but is not assigned a site.  That logic is very clever at predicting a user's intention.  It would be great to get the opinion of Linus and others who want multiple sites sharing the same name.<br><br></div><div>A couple of suggestions relating to the drop-down list:<br></div><div>(1) get rid of the space between the ~ and the distance.  I'm guessing a trivial fix.  E.g. ~365km away rather than ~ 365km away<br></div><div>(2) also provide the number of dives at the existing site to make it easy to tell the difference between the usual location (e.g. primary mooring) and the alternate location.  Or in my case, the dive site I'm using for all my other dives here, and the 'ghost' dive site that no longer has any dives assigned to it, but I haven't been able to delete from my log.  Hopefully not too hard to implement.  E.g. ~365km away, 14 previous dives</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
<br>
</span>Hmmm, summer, weekend, diving perspective ...  Not sure there will be<br>
a lot of testing ;-)<br>
<br>
The patch worked for me too.  Thanks Tomaz.<br>
There is an annoying thing i didn't realize yesterday (or may be it<br>
wasn't there):<br>
While typing, and a word is completed, the drop down list closes and<br>
your typing is stucked for a moment. When a new word beggins the list<br>
opens again and so on.<br></blockquote><div><br></div><div>I noticed this too, but only after reading Salvador's email.  Until reading his email I hadn't bothered completing words, and just selected the site I wanted before I had finished typing the first word.  Hopefully it isn't too hard to fix.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Globally, I think it will be a great functionality once "Manage dive<br>
sites" gets working.<br>
<br></blockquote><div><br></div><div>Yes, I'm patiently waiting for this feature too.  I've cleaned up my log and am deliberately not using duplicate locations for the same site, but it's not possible to delete the ghost sites that are left in my .ssrf file, except by opening the file in a text editor.  As well as the ability to:<br> add notes to a site, and reassign its coordinates (feature we had briefly with the pop-up dive site editor), I would love the option to:<br></div><div>(1) delete unused sites<br></div><div>(2) reassign coordinates with help of Marble (feature we had briefly with the pop-up dive site editor)<br></div><div>(3) add notes to a site  (feature we had briefly with the pop-up dive site editor)</div><div>(4) merge sites with close locations, regardless of name<br></div><div>(5) merge sites with close-ish locations and same or similar name (e.g. Lighthouse vs Lighthouse (Malapascua)).  I might have a GPS fix for one, whereas the other time I just guessed by clicking on the map but was actually hundreds of metres off.<br></div><div>(6) merge sites with same/similar names where one has GPS fix and the other doesn't.<br></div><div>(7) get georeferencing for all sites that don't already have it.<br><br></div><div>I think points (1) and (2) are crucial.  The rest are my wishlist.<br></div><div><br></div><div>For merging sites, I think the logic should be:<br></div><div>- if the sites name are different, let the user select the name they want for the merged site<br></div><div>- if one site has coordinate/notes and the other doesn't, take the data we have<br></div><div>- if both sites have (different) coordinates, let the user choose - ideally by showing both in marble and the user click on the desired site<br></div><div>- if both sites have notes, let the user choose one or the other, or take text from both.<br></div><div><br></div>Cheers,<br><br></div><div class="gmail_quote">Rick<br></div><br></div></div></div>