Hi,

Thanks indeed for the feedback. I think not quite all are bugs, but see more details inside.


> On Jan 14, 2016, at 12:59 PM, Andrei Chis <chisvasileandrei@gmail.com> wrote:
>
> Hi Nicolai,
>
> Thanks for reporting these issues. They are indeed bugs and we are working on fixing them.
>
> Cheers,
> Andrei
>
> On Tue, Jan 12, 2016 at 11:24 PM, Nicolai Hess <nicolaihess@gmail.com> wrote:
>
>
> 2016-01-08 11:24 GMT+01:00 Tudor Girba <tudor@tudorgirba.com>:
> Hi,
>
> We are about to integrate in Pharo a new member of the Glamorous Toolkit: the GTDebugger. As this is a significant change that might affect your workflow, here is some background information to help you deal with the change.
>
> First, you should know that the change is not irreversible and it is easily possible to disabled the new debugger through a setting. However, please do take the time to provide us feedback if something does not work out for you. We want to know what can be improved and we try to react as fast as we can.
>
> A practical change comes from the fact that the variables are manipulated through a GTInspector, which makes it cheaper to maintain in the longer run.
>
> Accept and Cancel buttons shouldn't be there
> or should not act on if the codepane hasn't changed.
> (every press on "accept" writes a new method version, although the contents didn't changed - tested on
> Latest update: #50524 )

Good catch. This will happen even if the button would not be present.


> Most (all?) other tools don't have Accept/Cancel buttons.

The logic is that these are actions that do not depend on the selection, so in Glamour we map these on actions that are applicable to the entire presentation. A similar approach is present in the inspector, although probably it does not appear so prominently because there is no text. We could try to add them in a dropdown menu. Would that help?


> - I really miss the "List Methods using 'varname'/List Methods storing into ‘varname'

Please open an issue for this.

> - is "stackTop" now gone ? I thought you wanted to add it to the stack ?
> - thisContext is gone as well ?

These were removed due to emergency mail exchange that happened during the last weekend.


> - the Bytecode/GT button is badly placed, it looks like the "downarrow" window menu icon
>   is a dropdown menu with label “Bytecode"

We could try to fix by adding more space to the left of the menu bar in the theme. Could you open an issue?


> (since when do we put buttons in the title pane?

Since Glamour makes it easy to have them there :). The GTPlayground and GTInspector have them, too.

> - the evaluator pane is shown as "dirty", as it does not make a difference if we
>   accept the text in this pane, there shouldn't be a dirty indicator.

This is a problem that seems to exist with basically every pharoScript in Glamour. You commented on an issue for this:
https://pharo.fogbugz.com/f/cases/16757/GLMHighlighterTextRubEditingMode-always-indicates-text-was-changed-even-when-it-wasn-t

> - you can not use the inspector pane to change inst var values

This is a problem but it is not specific to the debugger.


> - there is no way to refresh the inspector pane

Indeed. Would it be enough if we added a refresh for the whole debugger?

Cheers,
Doru


> I don't open bugtracker entries now, I 'll wait maybe this issues aren't bugs but
> features.
>
>
> nicolai
>
>
>
> _______________________________________________
> Moose-dev mailing list
> Moose-dev@list.inf.unibe.ch
> https://www.list.inf.unibe.ch/listinfo/moose-dev

--
www.tudorgirba.com
www.feenk.com

"Not knowing how to do something is not an argument for how it cannot be done."