<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">On Tue, 26 May 2015 23:32 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"><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>
_______________________________________________<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>