2016-01-15 13:39 GMT+01:00 Tudor Girba <tu...@tudorgirba.com>:

> 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 <chisvasileand...@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 <nicolaih...@gmail.com>
> wrote:
> >
> >
> > 2016-01-08 11:24 GMT+01:00 Tudor Girba <tu...@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.
>

This issue is one year old:
14583
<https://pharo.fogbugz.com/f/cases/14583/inspector-and-instvars-list-methods-storing-into-using>
inspector and instvars : list methods storing into/using


>
>
> > - 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.
>

still I don't think this is a good idea. What is the difference between
action buttons in the title pane
and the other one.


>
>
> > - 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


Yes this is working nwo in 3.8.


>
>
>
>
> > - 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?
>

I don't know if there is an easy solution. But it was nice in the old
debugger resp. inspector pane to have auto-updates for this values.


>
> 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-...@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."
>
>
>

Reply via email to