
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Easy to spot the issue. ShiftTime::reddit() updates the dive, which updates the UI, which triggers the on_dateEdit_dateChanged() signal, which calls shiftTime() again...</div><div class=""><br class=""></div><div class="">Berthold, is this something that you could poke at? trivial to reproduce, and I don't want to mess with your logic in that code too much :-)</div><div class=""><br class=""></div><div class="">Thanks</div><div class=""><br class=""></div><div class="">/D</div><br class=""><div><blockquote type="cite" class=""><div class="">On Jun 23, 2019, at 12:44 PM, Dirk Hohndel <<a href="mailto:dirk@hohndel.org" class="">dirk@hohndel.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Indeed, that's broken.<br class="">I'll take a look<br class=""><br class="">/D<br class=""><br class=""><div class="gmail_quote">On June 24, 2019 2:02:27 AM GMT+08:00, Patrick Valsecchi <<a href="mailto:patrick@thus.ch" class="">patrick@thus.ch</a>> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">Hi<br class=""><br class="">I see another problem in this area of the soft. Changing the date is not <br class="">working:<br class=""><br class="">1) Log -> Add dive<br class=""><br class="">2) Click on the date field (defaults to today's date: Sun Jun 23, 2019), <br class="">this displays a calendar<br class=""><br class="">3) Click on the 22nd<br class=""><br class="">4) The resulting date will be Wed Jun 26, 2019<br class=""><br class="">It's not easy to pick the correct date that way ;-)<br class=""><br class="">Tested with 4b0f90ced direct from the oven.<br class=""><br class="">CU<br class=""><br class="">On 22.06.19 21:33, Dirk Hohndel wrote:<br class=""><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">On Sat, Jun 22, 2019 at 09:14:45PM +0200, Berthold Stoeger wrote:<br class=""><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">1) See attached image. The default profile of 15m for 45 minutes shows a<br class="">crazy profile that is not useful to anybody. The descent rate to 15m<br class="">should be in the order of 20m/min, NOT 15m/22min. Similarly the default<br class="">ascent rate should be around 10m/min. The profile should have a square<br class="">shape, not a triangular shape.<br class=""></blockquote>That is indeed a bug. Weirdly I get a different profile than what you show in<br class="">the picture. But when I enter 45 in the duration box a sensible profile<br class="">appears.<br class=""></blockquote> What exactly you get depends in part on your settings when you last used<br class=""> the planner. I just sent out a pull request that stops using the planner<br class=""> to set up that initial dive profile and I think that will address this<br class=""> problem in a reasonably clean way.<br class=""><br class=""><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">2) The normal red cross-haired cursor and the white points on the<br class="">vertices of the profile are gone so that the default profile cannot be<br class="">edited on the profile panel. See section 5.1.1 in user manual.<br class=""></blockquote>This is a consequence of the half-baked undo system. To edit the profile you<br class="">now have to right click on the profile and select "edit profile". This will go<br class="">into the remnants of the dive edit mode. Perhaps a button should appear for<br class="">manually entered dives?<br class=""></blockquote> For people used to the user experience in earlier Subsurface versions this<br class=""> may be a bit unintuitive, but once you used the new method once I think<br class=""> it's really straight forward. I don't think this deserves another UI<br class=""> element.<br class=""><br class=""><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">Do you think that editing the profile is the common use case? I would rather<br class="">think that people simply enter date, depth and time, isn't it?<br class=""></blockquote> This one is interesting. We initially thought that people would prefer the<br class=""> graphical dive profile entry and some definitely did. But we got a lot of<br class=""> requests to add the text input fields (which initially we didn't have -<br class=""> all we had was moving the way points in the profile).<br class=""><br class=""><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">In any case, after adding a dive, the first field, "Date", should be selected.<br class="">And depth / duration should probably come before the temperature fields...<br class=""></blockquote> As we discussed - there is definitely room for improvement in the way we<br class=""> organize / display information on the screen.<br class=""><br class=""> /D<hr class=""> subsurface mailing list<br class=""> <a href="mailto:subsurface@subsurface-divelog.org" class="">subsurface@subsurface-divelog.org</a><br class=""> <a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" class="">http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface</a><br class=""></blockquote></pre></blockquote></div><br class="">-- <br class="">From my phone</div>_______________________________________________<br class="">subsurface mailing list<br class=""><a href="mailto:subsurface@subsurface-divelog.org" class="">subsurface@subsurface-divelog.org</a><br class="">http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface<br class=""></div></blockquote></div><br class=""></body></html>