Select location of divesite in new map widget

[Stefan Fuchs]

Hi Lubomir,

Am 03.10.2017 um 20:40 schrieb Stefan Fuchs:
> Am 03.10.2017 um 19:16 schrieb Lubomir I. Ivanov:
>> On 3 October 2017 at 20:04, Stefan Fuchs <sfuchs at gmx.de> wrote:
>>> [...]
>>>
>>> a tooltip should be shown in the lines of "move the marker", unless
>>> Stefan has found a bug that is.
>>>
>>> Ah, perfect. Got it. The "new" thing is that I have to press the small "edit
>>> dive site" button. Then I get a new marker and I can move it around. Ok,
>>> makes sense.
>>>
>>> But... now I have a crash. Not sure if it is coming from new map widget
>>> implementation or Dirks latest "dive site country" thing (yes, I merged PR
>>> #626 "Country taxonomy").
>>> What happens is: I press the small "edit dive site" button for a dive which
>>> has NO dive site name yet. Then I do NOT enter a dive site name but either
>>> press "Apply changes" or "Discard changes" --> Crash
>>> It seams to be also the same situation if I enter some text and move the
>>> marker...
>>>
> Ok, my crash has nothing to do with all this "country" stuff.
> I created here a test version w/o both Tomaz (already in master) and
> Dirks (PR #626) "country" commits.
>
> I still see the crash. And I have further info:
> The crash only happens if the dive where I do "edit dive site" is
> inside a dive trip. Please see the example file attached which is good
> enough on my side to reproduce the issue.
> Again step by step:
> - Open Test.xml
> - Select dive 710
> - Do "edit dive site"
> - Press "Discard changes" either with or w/o editing s.th.
> --> Crash
I got a little bit lost yesterday - too much traffic:
Did you confirm the issue I described above? Is it similar to
https://github.com/Subsurface-divelog/subsurface/issues/633 ? Or
different thing?

Best regards
Stefan
-- 

Stefan Fuchs
E-Mail: sfuchs at gmx.de <mailto:sfuchs at gmx.de>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20171004/7ed33fbc/attachment.html>