<p dir="ltr">The idea of a ( non editable? ) combobox on the info tab for choosing the active dive site sounds like a simpler idea. <br>
As it is now, I can't get the thing to work. I've resorted to adding dives without a site and then adding the site id manually in the XML file. Otherwise, no matter what I've tried, I simply can't add a site.</p>
<br><div class="gmail_quote">On Wed, 27 May 2015 06:08 Tomaz Canabrava <<a href="mailto:tcanabrava@kde.org">tcanabrava@kde.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">It's up to the discussion actually. Dirk wants it to do both things, I think it should do just one thing ( manage the dive site )  and that changing the dive site should be done on a combobox in the info menu, or a mix of  both. </p>
<br><div class="gmail_quote">On Tue, May 26, 2015, 23:55 Benjamin <<a href="mailto:nystire@gmail.com" target="_blank">nystire@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">If that is the case, maybe move the "manage dive sites" button into the top menu? As it currently stands, it gives the impression that it controls the dive site for the current dive.</p>
<p dir="ltr">Benjamin</p>
<br><div class="gmail_quote"></div><div class="gmail_quote">On Tue, 26 May 2015 23:32 Tomaz Canabrava <<a href="mailto:tcanabrava@kde.org" target="_blank">tcanabrava@kde.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div dir="ltr" style="text-align:left">both </div></div><div><div dir="ltr" style="text-align:left">I think I need to discuss something with you, regarding the dive site edit</div></div><div><div dir="ltr" style="text-align:left">it seems here that most of the stuff that I did 'wrong' is because we are thinking about different things for it.</div></div><div><div dir="ltr" style="text-align:left">for instance, you set the curr_dive_site to be the same as displayed_dive.dive_site</div></div><div><div dir="ltr" style="text-align:left">and I was programming as if the curr_dive_site is the current one selected from the combobox</div></div><div><div dir="ltr" style="text-align:left">the 'manage' button shouldn't change the dive sites of the current dive, but just manage the avaliable dive_sites.</div></div><div><div dir="ltr" style="text-align:left">and to change the dive_site of the current dive, you would change in the information panel, as any other information.</div></div><div><div dir="ltr" style="text-align:left">what you think?</div></div></div></div></div></blockquote></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
_______________________________________________<br>
subsurface mailing list<br>
<a href="mailto:subsurface@subsurface-divelog.org" target="_blank">subsurface@subsurface-divelog.org</a><br>
<a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" target="_blank">http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div>