<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 31, 2014 at 10:45 AM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Oct 30, 2014 at 04:22:03PM -0400, John Van Ostrand wrote:<br>
> I was getting very frustrated adding a dive location to an existing dive. I<br>
> was expecting to be able to add a location then select the coordinates from<br>
> the globe and have the coordinates updated and the location name and<br>
> coordinates associated. Neither happens. The flag is planted without text<br>
> and the flag disappears when I save.<br>
<br>
</span>That's not how it's supposed to work :-(<br></blockquote><div><br></div><div>It's not supposed to work the way I expected or the way it does?<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> I thought that commit 9ead871d6456f8f19f6f0fe2413513ef4449253d was the<br>
> culprit except reverting it seemingly made no difference.<br>
> (qt-ui/globe.cpp:135 is the effective area of that commit.)<br>
><br>
> So I added a dive to see what functionality that commit was supposed to<br>
> correct. By double clicking, the coordinates are associated with the<br>
> location of the highlighted dive in the dive list. I can see why that's bad.<br>
><br>
> Then restoring the commit gave me the same add-dive bug, odd but I'll<br>
> forget about that right now.<br>
<br>
</span>You lost me.<br></blockquote><div><br></div><div>It looked as if that patch may have fixed a bug while adding a dive but created one while editing a dive. I tested but the edit-dive problem was elsewhere and I couldn't see a difference in how add-dive worked so I'm not sure what bug it's supposed to have fixed. I don't think that's important right now though.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> So I'm looking for ideas on proper behaviour. There are several<br>
> circumstances I can think of:<br>
><br>
> 1. User is adding a dive.. Associate the location with the edit location<br>
> name of the dive.<br>
> 2. User is editing an existing dive. Associate the location with the edit<br>
> location name of the dive.<br>
<br>
</span>I don't know what you mean by "edit location name of the dive" in both of<br>
these cases.<br></blockquote><div><br></div><div>I've looked at the code so I can write a little more concisely about it.<br><br></div><div>1 and 2. When adding or editing a dive dive double clicking on the globe should add coordinates to the Coordinates field and place a marker on the globe so the user can see that he selected appropriate coordinates *before saving*. Neither happens, right now the action has no effect on the dive.<br><br>I've been able to make changes so the ui.coordinates field is updated but placing flags on the globe is more challenging. Since the edited location name and coordinates only exist in the ui.location and ui.coordinates variables the globe::repopulateLabels() function can't place a flag on the globe, repopulateLabels() uses the dive list for coordinates and labels. Adding code so it also checks isEditing() and places a flag based on the edited fields would add that functionality.<br><br></div><div>I had assumed a list associating location names and coordinates was kept which is why I wrote "associate the location [i.e. coordinates] with the location name."<br></div><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> 3. User is not editing a dive. Associate the location with the recorded<br>
> location name of the dive.<br></span></blockquote><div><br></div><div>Circumstance 3 works as expected.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
</span>Yep.<br>
<span class=""><br>
> The problem arises that if the user changes the location while editing, or<br>
> cancels the edit that needs to change the location-coordinate mapping.<br>
<br>
</span>Sorry, your language is just too confusing for me. There are three things<br>
that we need to treat separately:<br>
- location name<br>
- coordinates<br>
- flag placement on the globe<br>
<br>
Can you rephrase your three cases and your issue using these terms? Then<br>
we can determine which scenario isn't working right and hopefully fix<br>
it...<br>
<span class=""><br>
> So, do we track that in the edit session and update it on a name change or<br>
> revert it on cancel?<br>
><br>
> Or do we inform the user, on a double click, that they need to save first?<br>
<br>
</span>So this is the one part I did understand. And I responded to THAT in my<br>
previous email.<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div>John Van Ostrand<br></div><div>At large on sabbatical<br></div><br></div></div>
</div></div>