<div dir="ltr"><div>I was getting very frustrated adding a dive location to an existing dive. I was expecting to be able to add a location then select the coordinates from the globe and have the coordinates updated and the location name and coordinates associated. Neither happens. The flag is planted without text and the flag disappears when I save.<br><br>I thought that commit 9ead871d6456f8f19f6f0fe2413513ef4449253d was the culprit except reverting it seemingly made no difference. (qt-ui/globe.cpp:135 is the effective area of that commit.)<br><br></div><div>So I added a dive to see what functionality that commit was supposed to correct. By double clicking, the coordinates are associated with the location of the highlighted dive in the dive list. I can see why that's bad.<br></div><div><br></div><div>Then restoring the commit gave me the same add-dive bug, odd but I'll forget about that right now.<br><br></div><div>So I'm looking for ideas on proper behaviour. There are several circumstances I can think of:<br><br></div><div>1. User is adding a dive.. Associate the location with the edit location name of the dive.<br></div><div>2. User is editing an existing dive. Associate the location with the edit location name of the dive.<br></div><div>3. User is not editing a dive. Associate the location with the recorded location name of the dive.<br><br>The problem arises that if the user changes the location while editing, or cancels the edit that needs to change the location-coordinate mapping.<br></div><div><br></div><div>So, do we track that in the edit session and update it on a name change or revert it on cancel?<br><br>Or do we inform the user, on a double click, that they need to save first?<br></div><div> <br>-- <br><div dir="ltr"><div>John Van Ostrand<br></div><div>At large on sabbatical<br></div></div></div></div>