On Sat, May 24, 2014 at 1:02 PM, Dirk Hohndel <[email protected]> wrote:
> On Sat, May 24, 2014 at 11:49:41AM -0300, Tomaz Canabrava wrote:
>> On Sat, May 24, 2014 at 11:42 AM, Dirk Hohndel <[email protected]> wrote:
>> > On Fri, May 23, 2014 at 10:44:23PM -0300, Tomaz Canabrava wrote:
>> >> This makes the add planer USABLE. <3
>> >> behold the cuteness overload of the new profile acting as the Add Widget 
>> >> canvas.
>> >
>> > Awesome work, Tomaz.
>> >
>> > I played with it quite a bit and find the resizing action very neat - it's
>> > a bit jumpy and takes some getting used to, but it's WAY better than the
>> > clumsy buttons to increase / decrease depth and time that we had before!
>>
>> The jumpyness is because we are creating a dive from the plan
>> everytime a move is done,
>
> Should we do this differently? In order to get the deco stuff right, I
> don't think we can do this differently. But maybe we can be smarter about
> telling the profile what has to change?

We can - but then, we will lose some stuff - mostly the colors of the
velocities on the polygon.
( that's what I meant by color border below ), since we are showing an
actual dive, everything
on the profile is a real dive, so the velocity colors on the borders
and other stuff is ready.

if this thing is not important, I can speed up quite a lot by using
another model to display stuff
 instead of the dive profile model for that.

>> there's a bug where the lines are comming from the wrong direction too
>> - they should come from
>> the bottom / from the right, but are commiing from the top and left,
>> making the profile
>> do more work than it should, but it's a simple fix.
>
> I hadn't noticed that.
>
>> > One thing that I really wonder about is the tooltip / information overlay.
>> > Is that useful in dive add / dive plan mode?
>>
>> It is - because the dive that the user is seeing on the information
>> overlay is the same that
>> will be the dive when he accept - did you see that the colors of the
>> borders are colored ?
>> its a live preview. :)
>
> Not sure what you mean with the colors of the borders...
> What I do notice is that the screen is all blue (instead of cream) - I
> guess that's OK as it indicates we are in create dive mode.
> The other thing that I noticed is that if I cancel "Add Dive" the profile
> of the selected dive is not restored. I have to switch dives to get the
> profile back.

'cancel' and 'ok' are a bit bugged, but on my todo list. :)
 I'm trying to make the profille really workable, then I'll fix the rest.

>> >> Keyboard handling is broken because I didn't ported *anything* of it yet.
>>
>> This is not true anymore, just ported the keyboard handling. :P
>
> I saw more patches. Will get to them next :-)
>
>> > The other thing we need to figure out is how we address the very real user
>> > feedback that we alternatively need to allow people to enter data directly
>> > and have us create a reasonable profile for that.
>>
>> The same way that we allowed in the past but it was removed, there's
>> already a widget that allows that - but currently only for planner
>> mode.
>
> We need to integrate this in a consistent way into add dive as well.

we do. it can be a tab on the info panel.

>
> /D
_______________________________________________
subsurface mailing list
[email protected]
http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to