August 2016 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] <example>/<examplar>

by Dimitris Chloupis

I will probably sound stupid but here I go. I am no fan of Pragmas. I feel that their syntax is weird and unnecessary and what they do can be easily achieved with a secondary class that provides this meta data. In case of examples it would have been possible to have examples in a separate package together with a class that provides the additional info the IDE needs to trigger the right tools. On the other hand I was neither a fan of using example as part of a method name. Saying that instancePrototype sounds a lot more clear than examplar. On Mon, 22 Aug 2016 at 08:39, stepharo <stepharo(a)free.fr> wrote: > Hi sean > > this is exactly with <sampleInstance> but this is not exactly the > solution doru envision. > > But I think that sampleInstance is letting people build extra tools and > also let people use the sample method > > in their test and others. > > > Stef > > > Le 21/8/16 Ã 02:01, Sean P. DeNigris a Ã©crit : > > Tudor Girba-2 wrote > >> In this case, you will not be able to use the resulting object, and the > >> new energy around examples started from the need to utilize that ability > >> The other solution is to delegate the action to another pragma that can > >> complement the example one. > > Isn't that what we've settled on? That is, <sampleInstance> for when you > > need the result? It seems that all bases are covered if those methods > return > > an instance and the browser button takes that instance and opens an > > inspector, no? > > > > > > > > ----- > > Cheers, > > Sean > > -- > > View this message in context: > http://forum.world.st/example-examplar-tp4911728p4912083.html > > Sent from the Pharo Smalltalk Developers mailing list archive at > Nabble.com. > > > > > > >

Aug. 23, 2016

Re: [Pharo-dev] About PythonDocTest for Pharo

by Nicolas Passerini

On Thu, Aug 18, 2016 at 11:28 PM, Denis Kudriashov <dionisiydk(a)gmail.com> wrote: > > 2016-08-18 13:32 GMT+02:00 stepharo <stepharo(a)free.fr>: > >> withExtension: aString >> >> "Returns a new file reference with a different file extension" > > > Does this comment is really needed? > I only see extra word "different". And rest is same as method name. IMHO > we should rename method to "withDifferentExtension:" and remove comment. > I think that it is interesting that the comment also says that it will return a FileReference... of course there are other ways to know that, but it is nice that the comment avoids the research/guessing. More over, which is not obvious is that it will *change* the extension, because the "with" prefix could mislead you to think that it will *add* the extension. Maybe we can come up with a better name (withDifferentExtension: works better, I agree) or we need the comment to be explicit about it.

Aug. 23, 2016

Re: [Pharo-dev] Character encoding and line ending in OSSubprocess

by Nicolas Passerini

On Mon, Aug 22, 2016 at 4:29 PM, Thierry Goubier <thierry.goubier(a)gmail.com> wrote: > > > 2016-08-22 16:08 GMT+02:00 Nicolas Passerini <npasserini(a)gmail.com>: > >> >> On Mon, Aug 22, 2016 at 3:31 PM, Thierry Goubier < >> thierry.goubier(a)gmail.com> wrote: >> >>> does this happens when you are using one of the GitFileTree generated >>> command or when you use directly the command line api (runGitCommand and >>> friends)? >>> >> >> I think that it happens in both cases, the most obvious is that it fails >> to bring my github name, which happens to have an "Ã¡". >> > > Hum, I may be missing a string convert on some arguments of the git clone > command... > > Ok, if that's the way to go I can try to fix it and send you a pull request. I was thinking if it can not be resolved by OSSubprocess itself. > But I will check your idea, to be sure. >> >> >>> Does this happens with OSProcess as well? >>> >>> I am not sure about this, can I use gitfiletree with OSProcess, how can >> I test it? >> >> > Just load OSProcess before loading GitFileTree, it should notice it and > avoid loading OSSubprocess. > > > Ok, I will try that, thankyou.

Aug. 23, 2016

Re: [Pharo-dev] Usability issue : the class/instance button in Pharo 5 gives poor feedback

by Ben Coman

On Tue, Aug 23, 2016 at 4:36 AM, Stephan Eggermont <stephan(a)stack.nl> wrote: > On 20/08/16 16:33, Oscar Nierstrasz wrote: >> >> >> Well, as a user I want to see the current context as well. >> Thanks for pointing out that the methods and protocols on the class side >> are bold! > >>Now that I know this I will watch for it; but it is not at all obvious. > > Indeed. A visual indicator not only needs to be there, it also needs to be > obviously connected to the widget influencing it. The bottom of the protocol pane would be a good option. An alternative would be the protocol pane being a tree view with two top level items . * Instance protocols * Class protocols Actually!! this might have the added benefit of helping newcomers. It seems common that newcomers following tutorials lose track of which "side" they are adding methods too, which may(?) lead to frustration and abandoning of the tutorial. cheers -ben > Neither of the three > buttons provide that. The hover texts should adapt to the button state, > b.t.w. > > I thought the color differences of the forward and backward arrow of the > history navigator might have something to do with being on the first/last > item, but that is not the case. > > Stephan > >

Aug. 23, 2016

Re: [Pharo-dev] Usability issue : the class/instance button in Pharo 5 gives poor feedback

by Henrik Nergaard

What about something like this? Best regards, Henrik -----Original Message----- From: Pharo-dev [mailto:pharo-dev-bounces@lists.pharo.org] On Behalf Of Stephan Eggermont Sent: Monday, August 22, 2016 10:36 PM To: pharo-dev(a)lists.pharo.org Subject: Re: [Pharo-dev] Usability issue : the class/instance button in Pharo 5 gives poor feedback On 20/08/16 16:33, Oscar Nierstrasz wrote: > > Well, as a user I want to see the current context as well. >Thanks for pointing out that the methods and protocols on the class side are bold! >Now that I know this I will watch for it; but it is not at all obvious. Indeed. A visual indicator not only needs to be there, it also needs to be obviously connected to the widget influencing it. Neither of the three buttons provide that. The hover texts should adapt to the button state, b.t.w. I thought the color differences of the forward and backward arrow of the history navigator might have something to do with being on the first/last item, but that is not the case. Stephan

Aug. 22, 2016

Re: [Pharo-dev] Usability issue : the class/instance button in Pharo 5 gives poor feedback

by Stephan Eggermont

On 20/08/16 16:33, Oscar Nierstrasz wrote: > > Well, as a user I want to see the current context as well. >Thanks for pointing out that the methods and protocols on the class side are bold! >Now that I know this I will watch for it; but it is not at all obvious. Indeed. A visual indicator not only needs to be there, it also needs to be obviously connected to the widget influencing it. Neither of the three buttons provide that. The hover texts should adapt to the button state, b.t.w. I thought the color differences of the forward and backward arrow of the history navigator might have something to do with being on the first/last item, but that is not the case. Stephan

Aug. 22, 2016

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/60190 Home: https://github.com/pharo-project/pharo-core

Aug. 22, 2016

[pharo-project/pharo-core] d0c90b: 60190

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: d0c90b840f6f8f5644e97e5cf4a8159ca9cfc712 https://github.com/pharo-project/pharo-core/commit/d0c90b840f6f8f5644e97e5c… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-08-22 (Mon, 22 Aug 2016) Changed paths: M Nautilus.package/AbstractMethodIconAction.class/instance/order/isActionHandled.st A Nautilus.package/AbstractNautilusUI.class/class/icons behavior/runExamplarMethod_.st M Nautilus.package/AbstractNautilusUI.class/instance/menus behavior/restrictedBrowseGroups_.st A Nautilus.package/MethodSampleInstanceAction.class/README.md A Nautilus.package/MethodSampleInstanceAction.class/definition.st A Nautilus.package/MethodSampleInstanceAction.class/instance/order/actionOrder.st A Nautilus.package/MethodSampleInstanceAction.class/instance/order/isActionHandled.st A Nautilus.package/MethodSampleInstanceAction.class/instance/order/privateActionIcon.st A Nautilus.package/extension/Group/instance/restrict_.st A Nautilus.package/extension/MethodGroup/instance/restrict_.st A Nautilus.package/extension/PackageGroup/instance/restrict_.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60189.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60190.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60189.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60190.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 60190 18972 Group browse scoped class sync https://pharo.fogbugz.com/f/cases/18972 18964 support for <sampleInstance> https://pharo.fogbugz.com/f/cases/18964 http://files.pharo.org/image/60/60190.zip

Aug. 22, 2016

Re: [Pharo-dev] Usability issue : the class/instance button in Pharo 5 gives poor feedback

by stepharo

I will open an issue and try to allocate time to address but if someone has some time it would be great to have some helps. Le 22/8/16 Ã 14:37, Alistair Grant a Ã©crit : > On Sun, Aug 21, 2016 at 09:30:59AM +0200, stepharo wrote: >> If you have ideas let us know. >> >> In the past we changed for the radio because of that. >> >> What I do not like that is that we cannot press on a package when we are in >> hierarchy mode >> >> and this is super boring because it breask my flow. > +1 > >> >> Stef >

Aug. 22, 2016

Re: [Pharo-dev] Character encoding and line ending in OSSubprocess

by Thierry Goubier

2016-08-22 16:08 GMT+02:00 Nicolas Passerini <npasserini(a)gmail.com>: > > On Mon, Aug 22, 2016 at 3:31 PM, Thierry Goubier < > thierry.goubier(a)gmail.com> wrote: > >> does this happens when you are using one of the GitFileTree generated >> command or when you use directly the command line api (runGitCommand and >> friends)? >> > > I think that it happens in both cases, the most obvious is that it fails > to bring my github name, which happens to have an "Ã¡". > Hum, I may be missing a string convert on some arguments of the git clone command... > But I will check your idea, to be sure. > > >> Does this happens with OSProcess as well? >> >> I am not sure about this, can I use gitfiletree with OSProcess, how can I > test it? > > Just load OSProcess before loading GitFileTree, it should notice it and avoid loading OSSubprocess. Thierry

Aug. 22, 2016