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.
On Tue, May 26, 2015, 23:55 Benjamin <[email protected]> wrote: > 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. > > Benjamin > > On Tue, 26 May 2015 23:32 Tomaz Canabrava <[email protected]> wrote: > >> both >> I think I need to discuss something with you, regarding the dive site edit >> it seems here that most of the stuff that I did 'wrong' is because we are >> thinking about different things for it. >> for instance, you set the curr_dive_site to be the same as >> displayed_dive.dive_site >> and I was programming as if the curr_dive_site is the current one >> selected from the combobox >> the 'manage' button shouldn't change the dive sites of the current dive, >> but just manage the avaliable dive_sites. >> and to change the dive_site of the current dive, you would change in the >> information panel, as any other information. >> what you think? >> > _______________________________________________ >> subsurface mailing list >> [email protected] >> http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface >> >
_______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
