- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Missing refresh after method definition

by Thierry Goubier

Le 28/03/2017 Ã 14:59, phil(a)highoctane.be a Ã©crit : > > On Tue, Mar 28, 2017 at 2:44 PM, Thierry Goubier > <thierry.goubier(a)gmail.com <mailto:thierry.goubier@gmail.com>> wrote: > > Le 28/03/2017 Ã 14:31, Cyril Ferlicot D. a Ã©crit : > > On 28/03/2017 12:39, Stephane Ducasse wrote: > > Hi > > I often defined method and let the system guess and propose > the addition > to new instance variables and (they are red) but once I > created the > instance variables they should not be red anymore. Now > nautilus does not > refresh the pane and I end up having a warning about a non > defined > instance which is not one. > Did you see this glitch? > > Stef > > > I opened an issue last year: > https://pharo.fogbugz.com/f/cases/19150/Generating-new-instance-variable-co… > <https://pharo.fogbugz.com/f/cases/19150/Generating-new-instance-variable-co…> > > > Good point. > > Need to refresh code view on a class method (and subclass methods) > when class reorganized event is sent... take care of not removing > unsaved changes then. Maybe toggling off and on syntax highlighting > may be enough. > > > There is also the change code in debugger, have it compiled but the > method is still the same on the next run. Are you sure? Each time I modify and save in the debugger, the system browser view of the method is updated as well. Thierry > > Phil > > > > > Thierry > >

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by phil＠highoctane.be

On Tue, Mar 28, 2017 at 2:59 PM, Denis Kudriashov <dionisiydk(a)gmail.com> wrote: > > 2017-03-28 14:38 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: > >> I want a "remove all of the yellowish tempvars from this method" >> refactoring then. > > > Generally we need more intelligence on critic icons. Now they not allow > "fix it" action. > I'd add that we need to segregate "navigation icons" from "indication icons" from "action icons" (e.g. overriden/overriding method vs test pass vs run test/initialize). And also allow for more than one action on a browser entry (method/protocol/... etc). The mechanism for extension is powerful but is crippled by the inability to have more than one thing (e.g. I would like to have support for several pragmas or name matches). These should at least appear as actions in the popup menu in a special section if we only allow a single top priority action on an item. This icon stuff proved problematic when refactoring some tests as there was no way to move up/down in the inherited methods because a green/red/gray bubble was on top (same for <script> BTW). This is super powerful and complements the GT ideas pretty neatly I think. I dream of Calypso meeting Nautilus and giving us the full feature set with clean code. Phil

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by phil＠highoctane.be

On Tue, Mar 28, 2017 at 2:44 PM, Thierry Goubier <thierry.goubier(a)gmail.com> wrote: > Le 28/03/2017 Ã 14:31, Cyril Ferlicot D. a Ã©crit : > >> On 28/03/2017 12:39, Stephane Ducasse wrote: >> >>> Hi >>> >>> I often defined method and let the system guess and propose the addition >>> to new instance variables and (they are red) but once I created the >>> instance variables they should not be red anymore. Now nautilus does not >>> refresh the pane and I end up having a warning about a non defined >>> instance which is not one. >>> Did you see this glitch? >>> >>> Stef >>> >> >> I opened an issue last year: >> https://pharo.fogbugz.com/f/cases/19150/Generating-new-insta >> nce-variable-could-refresh-the-highlight-of-the-code-editor >> > > Good point. > > Need to refresh code view on a class method (and subclass methods) when > class reorganized event is sent... take care of not removing unsaved > changes then. Maybe toggling off and on syntax highlighting may be enough. There is also the change code in debugger, have it compiled but the method is still the same on the next run. Phil > > > Thierry > >

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by Denis Kudriashov

2017-03-28 14:38 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: > I want a "remove all of the yellowish tempvars from this method" > refactoring then. Generally we need more intelligence on critic icons. Now they not allow "fix it" action.

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by Denis Kudriashov

2017-03-28 14:31 GMT+02:00 Cyril Ferlicot D. <cyril.ferlicot(a)gmail.com>: > I opened an issue last year: > https://pharo.fogbugz.com/f/cases/19150/Generating-new- > instance-variable-could-refresh-the-highlight-of-the-code-editor > I look at it and could not reproduce

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by Thierry Goubier

Le 28/03/2017 Ã 14:31, Cyril Ferlicot D. a Ã©crit : > On 28/03/2017 12:39, Stephane Ducasse wrote: >> Hi >> >> I often defined method and let the system guess and propose the addition >> to new instance variables and (they are red) but once I created the >> instance variables they should not be red anymore. Now nautilus does not >> refresh the pane and I end up having a warning about a non defined >> instance which is not one. >> Did you see this glitch? >> >> Stef > > I opened an issue last year: > https://pharo.fogbugz.com/f/cases/19150/Generating-new-instance-variable-co… Good point. Need to refresh code view on a class method (and subclass methods) when class reorganized event is sent... take care of not removing unsaved changes then. Maybe toggling off and on syntax highlighting may be enough. Thierry

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by phil＠highoctane.be

I want a "remove all of the yellowish tempvars from this method" refactoring then. I am finding myself clicking around and deleting these things too many times for my taste. Phil On Tue, Mar 28, 2017 at 2:30 PM, Denis Kudriashov <dionisiydk(a)gmail.com> wrote: > > 2017-03-28 14:23 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: > >> Yes I do see that. >> >> The system also does not removes the unused tempvars as it used to do in >> the past, they are now marked yellow: >> > > It is feature :) >

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by Cyril Ferlicot D.

On 28/03/2017 12:39, Stephane Ducasse wrote: > Hi > > I often defined method and let the system guess and propose the addition > to new instance variables and (they are red) but once I created the > instance variables they should not be red anymore. Now nautilus does not > refresh the pane and I end up having a warning about a non defined > instance which is not one. > Did you see this glitch? > > Stef I opened an issue last year: https://pharo.fogbugz.com/f/cases/19150/Generating-new-instance-variable-co… -- Cyril Ferlicot http://www.synectique.eu 2 rue Jacques PrÃ©vert 01, 59650 Villeneuve d'ascq France

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by Denis Kudriashov

2017-03-28 14:23 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: > Yes I do see that. > > The system also does not removes the unused tempvars as it used to do in > the past, they are now marked yellow: > It is feature :)

March 28, 2017

Re: [Pharo-dev] Missing refresh after method definition

by phil＠highoctane.be

Yes I do see that. The system also does not removes the unused tempvars as it used to do in the past, they are now marked yellow: Phil On Tue, Mar 28, 2017 at 12:39 PM, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: > Hi > > I often defined method and let the system guess and propose the addition > to new instance variables and (they are red) but once I created the > instance variables they should not be red anymore. Now nautilus does not > refresh the pane and I end up having a warning about a non defined instance > which is not one. > Did you see this glitch? > > Stef >

March 28, 2017