2016-01-15 13:39 GMT+01:00 Tudor Girba <tudor(a)tudorgirba.com>om>:
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(a)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(a)gmail.com>
wrote:
2016-01-08 11:24 GMT+01:00 Tudor Girba <tudor(a)tudorgirba.com>om>:
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-al…
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-dev(a)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."