- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] String>>#adaptToNumber:andSend:

by Marcus Denker

> On 26 May 2015, at 14:51, Florin Mateoc <fmateoc(a)gmail.com> wrote: > > Hi Stef, > > I think this one's easy, especially for Pharo. Since apparently it was Etoys, not Morphic, that was or is using the > strings and numbers mixed arithmetic, and Pharo does not care about Etoys, the arithmetic protocol (the operators plus > the adaptTo...:andSend: methods) should simply be removed from String. > While I did not perform any extensive testing, I did remove them in a Squeak (Morphic) image and nothing seemed to break. I committed a slice: https://pharo.fogbugz.com/f/cases/2906/Analyze-adaptToNumber-andSend <https://pharo.fogbugz.com/f/cases/2906/Analyze-adaptToNumber-andSend> Marcus

May 26, 2015

Re: [Pharo-dev] translateBy: vs translatedBy:

by Aliaksei Syrel

Hi Serge, I vote for renaming to Rectangle>>translatedBy:, because it returns a new translated rectangle instance. For example in bloc there is BlUserInputEvent and it has two methods: translateBy: and translatedBy:. First one actually modifies position (mutates object) the second one creates a translated copy of the event. Hence Rectangle is suppose to be immutable I think translatedBy: is better name. Cheers, Alex On Tue, May 26, 2015 at 2:41 PM, Serge Stinckwich < serge.stinckwich(a)gmail.com> wrote: > In Pharo 4.0, there is a Rectangle>>translateBy: aFactor method. > The same method in VW is called translatedBy: > > In Pharo, there is also a translatedToBeWithin: and > translatedAndSquishedToBeWithin: > and scaledBy: > > Just to be consistent, this is maybe better to use translatedBy: > instead of translateBy: ? > > -- > Serge Stinckwich > UCBN & UMI UMMISCO 209 (IRD/UPMC) > Every DSL ends up being Smalltalk > http://www.doesnotunderstand.org/ > >

May 26, 2015

Re: [Pharo-dev] String>>#adaptToNumber:andSend:

by Florin Mateoc

Hi Stef, I think this one's easy, especially for Pharo. Since apparently it was Etoys, not Morphic, that was or is using the strings and numbers mixed arithmetic, and Pharo does not care about Etoys, the arithmetic protocol (the operators plus the adaptTo...:andSend: methods) should simply be removed from String. While I did not perform any extensive testing, I did remove them in a Squeak (Morphic) image and nothing seemed to break. Even for Squeak, I think the solution should be the same and Etoys should be fixed to not use this particular hack. Florin On 5/26/2015 2:20 AM, stepharo wrote: > Hi florin > > I agree with you. If you have some cycles to propose a fix we would be delighted. > > Stef > > > > Le 25/5/15 19:24, Florin Mateoc a Ã©crit : >> On 5/25/2015 6:02 AM, stepharo wrote: >>> The problem is that while the intention was nice >>> 5 + (1 to: 10) could make sense, its side effects are worse. >>> >>> Stef >>> >>> >> >> >> I don't think there is anything wrong with your example, which is about vector operations and which I think make sense >> (I would have preferred if the result of the evaluation would have been an interval as well though). They are also part >> of the original protocol. >> >> I was only protesting the later addition of the arithmetic protocol to strings. It does not make sense to treat strings >> as vectors and there are no clear semantics for some other kind of string-specific arithmetic operations. >> >> Florin >> >>

May 26, 2015

translateBy: vs translatedBy:

by Serge Stinckwich

In Pharo 4.0, there is a Rectangle>>translateBy: aFactor method. The same method in VW is called translatedBy: In Pharo, there is also a translatedToBeWithin: and translatedAndSquishedToBeWithin: and scaledBy: Just to be consistent, this is maybe better to use translatedBy: instead of translateBy: ? -- Serge Stinckwich UCBN & UMI UMMISCO 209 (IRD/UPMC) Every DSL ends up being Smalltalk http://www.doesnotunderstand.org/

May 26, 2015

[pharo-project/pharo-core]

by GitHub

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

May 26, 2015

[pharo-project/pharo-core] 64e44a: 50066

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 64e44ae8b61daf57dac872e133a76ade74161327 https://github.com/pharo-project/pharo-core/commit/64e44ae8b61daf57dac872e1… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-05-26 (Tue, 26 May 2015) Changed paths: A Collections-Sequenceable.package/Interval.class/instance/accessing/sum.st M Nautilus.package/AbstractNautilusUI.class/instance/private/title.st M Nautilus.package/Nautilus.class/instance/browser compatibility/labelString.st M SUnit-Core.package/ClassFactoryWithOrganization.class/instance/creating/newClassNamed_subclassOf_instanceVariableNames_classVariableNames_category_.st M SUnit-Core.package/ClassFactoryWithOrganization.class/instance/creating/newSubclassOf_instanceVariableNames_classVariableNames_category_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50065.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50066.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50065.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50066.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st R Slot.package/OldClassBuilderAdapter.class/instance/adapted api/name_inEnvironment_subclassOf_type_instanceVariableNames_classVariableNames_poolDictionaries_category_.st Log Message: ----------- 50066 15611 Cleanup OldClassBuilderAdapter: remove last users of class creation method https://pharo.fogbugz.com/f/cases/15611 15608 Nautilus window title should include System Browser https://pharo.fogbugz.com/f/cases/15608 15610 Interval>>#sum is O(n) https://pharo.fogbugz.com/f/cases/15610 http://files.pharo.org/image/50/50066.zip

May 26, 2015

Re: [Pharo-dev] [Reflectivity] Progress for Breakpoints

by Sean P. DeNigris

Marcus Denker-4 wrote > I think they are more a trick for myself to do something tiny every day > that moves the project forward. Ha ha, whatever works! I find them interesting even if I don't totally understand and it's good to know that progress is being made. ----- Cheers, Sean -- View this message in context: http://forum.world.st/Reflectivity-Progress-for-Breakpoints-tp4825234p48286… Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

May 26, 2015

Re: [Pharo-dev] FogBugz (Case [Issue]13315) Morphic - NewList without icons

by Esteban Lorenzano

Hi, > On 24 May 2015, at 18:08, Nicolai Hess <nicolaihess(a)web.de> wrote: > > Thank your reply > > 2015-05-24 10:02 GMT+02:00 Esteban Lorenzano <estebanlm(a)gmail.com <mailto:estebanlm@gmail.com>>: > you can already play with it: > > Gofer it > smalltalkhubUser: 'estebanlm' project: 'FastTable'; > configuration; > loadStable. > > not in my place this weekend so I cannot finish it as I wanted before pushing it. > > My idea is to ask community help finish it. > > Implementation uses datasources to feed the list, so in theory you can browse infinite rows without loosing speed. > according to my tests (in my machine): > > [ Object browse ] timeToRun asMilliSeconds. -> ~3000ms (without cache) > [ Object browse ] timeToRun asMilliSeconds. -> ~350ms (with cache active) > [ Object browse ] timeToRun asMilliSeconds. -> ~150ms with FastTable, always. > > There is an adaptor FTPluggableIconListMorphAdaptor that has the same format as a PluggableIconListMorph so changing already existing calls will be easy (even if there should not be done like that in the long term :) > > So, so far, list works fine, but > > - still missing drag&drop support > - there are some issues remaining: > -- cells escaping his width > -- last elements in long lists sometimes are not visible > > some details has to be around too... but this is a good design, taken from Cocoa... is years better than the naive implementation of old PluggableListMorph and better than the strategy used by NewList (spawning threads all around). > > Some info about this? Is this in some mac developer guide or a paper written about the design behind cocoa? > there is some that I use as reference frequently: this one: https://developer.apple.com/legacy/library/documentation/Cocoa/Conceptual/C… <https://developer.apple.com/legacy/library/documentation/Cocoa/Conceptual/C…> is a âretiredâ document, but is retired because is old (old version of objective-c and old version of cocoa), but fundamentals remains ok. instead that document, apple refers now to here: https://developer.apple.com/library/mac/documentation/General/Conceptual/Co… <https://developer.apple.com/library/mac/documentation/General/Conceptual/Co…> last, I use as a source of inspiration this project: http://www.cocotron.org <http://www.cocotron.org/> which basically is a rewrite of cocoa but open source (and portable). Project is at least slow (if not stagnated), but I use it to consult algorithms and solutions they found to UI problems I find time to time. cheers, Esteban > > > Please, help me finish! > Repository is open so anybody can contribute. > > +1 > > > FTExamples has a set of wxamples with all things implemented :) > > Thank you for not only having examples but class comments too! > > > Esteban > > ps: I already has the adaptor working with Nautilus, but since it depends on some changes on it I will wait until is integrated to push it. But basically if you want to try you just need to go to: CategoryWidget>>#buildCategoriesList, ClassWidget>>#buildClassesList and MethodWidget>>#buildMethodList and change PluggableIconListMorph with FTPluggableIconListMorphAdaptor > > > > > On Sun, May 24, 2015 at 7:41 AM, stepharo <stepharo(a)free.fr <mailto:stepharo@free.fr>> wrote: > Hi nicolai > > Esteban is rewriting the list/table widgets and it is much much faster. > He should announce a beta any soon. > With it we will be able to remove > - NewList (because its implementation is not good) > - pluggableListMorph > - and potentially the paginated and morph tree morph. > > Stef > > > Le 23/5/15 00:20, Nicolai Hess a Ã©crit : >> Is there some more information about this? >> >> I opened that issue and I asks quesions about the NewList implementation on the mailing >> list and commented on some other issues about NewList (for example 13554 <https://pharo.fogbugz.com/default.asp?13554> Red cross in EyeDateAndTimeInspector). >> (With not so much feedback). >> And making NewListRender working with athens caused quite some headache to me. >> But this is the first time I hear someone is working on a new list. >> >> I would like to encourage people to make more comments on the bugtracker. >> >> And, as we have already quite some mess in our widget libraries, because we >> have all kinds of list/tree-morph-nomorph-pluggable-or-simple-with-or-without-icons-things, >> I would like some more discussion/background info about this. > > I agree. > I would like to remove them all and get only one >> >> Especially how we prevent to not have "just-another-list" in the image, but >> a way/plan to migrate or exchange the current list class to use the new/better one. > > Why the migration to NewList was stopped is that we realised that it was not good. > Alain told us that 8 months ago. > >> >> thanks >> nicolai >> >> >> >> >> >> ---------- Forwarded message ---------- >> From: Pharo Issue Tracker <do-not-reply(a)pharo.fogbugz.com <mailto:do-not-reply@pharo.fogbugz.com>> >> Date: 2015-05-22 11:23 GMT+02:00 >> Subject: FogBugz (Case [Issue]13315) Morphic - NewList without icons >> To: "nicolaihess(a)web.de <mailto:nicolaihess@web.de>" <nicolaihess(a)web.de <mailto:nicolaihess@web.de>> >> >> >> >> Marcus Denker resolved Case 13315: NewList without icons <https://pharo.fogbugz.com/default.asp?13315> and assigned it to you: >> >> Bug in Morphic: 1. Pharo Image >> >> We have a better list done by esteban soon >> >> Status changed: >> Work Needed Resolved (Invalid) >> >> Assigned To changed: >> Everyone Nicolai Hess >> >> Priority: 3 â Must Fix Status: Resolved (Invalid) >> Assigned to: Nicolai Hess Milestone: Pharo5.0 >> >> >> Go to Case >> >> <https://pharo.fogbugz.com/default.asp?13315> >> Don't want FogBugz notifications anymore? Update your preferences. <https://pharo.fogbugz.com/f/userPrefs> >> A case which you opened has been resolved ( Resolved (Invalid) ) by Marcus Denker in FogBugz. It is your responsibility to verify the resolution and close or reactivate the case. Case ID: 13315 Title: NewList without icons Status: Resolved (Invalid) Category: Bug Project: Morphic Area: 1. Pharo Image Priority: 3 - Must Fix Milestone: Pharo5.0: 4/1/2016 Assigned To: Nicolai Hess URL: https://pharo.fogbugz.com/default.asp?13315 Changes: Status changed from 'Work Needed' to 'Resolved (Invalid)'. Last message: We have a better list done by esteban soon If you do not want to receive automatic notifications anymore, change your preferences in the Options screen. (https://pharo.fogbugz.com/f/userPrefs) <>

May 26, 2015

Re: [Pharo-dev] FogBugz (Case [Issue]13315) Morphic - NewList without icons

by Esteban Lorenzano

Hi, > On 26 May 2015, at 10:37, Nicolai Hess <nicolaihess(a)web.de> wrote: > > > > 2015-05-24 10:02 GMT+02:00 Esteban Lorenzano <estebanlm(a)gmail.com <mailto:estebanlm@gmail.com>>: > > > - still missing drag&drop support > - there are some issues remaining: > -- cells escaping his width > -- last elements in long lists sometimes are not visible > > > Any special reasons why you omitted doubleClick and drag&drop ? Didn't it fit into the actual design or > is it ok to implement it the same way like it is in PluggableListMorph? > I didnât omit, just still did not implemented part of them: - double click is implemented, it announces a FTStrongSelectionChanged (and the examples show it). - double click is still not implemented for the PluggableEtcAdaptorâ¦ but this was lack of time, not by design :) - drag&drop is not implemented at all but again, because lack of time. I was targeting to work on it this week, but Iâm also full of other stuff, so you will made me happy if you took it :) cheers, Esteban

May 26, 2015

Re: [Pharo-dev] FogBugz (Case [Issue]13315) Morphic - NewList without icons

by Esteban Lorenzano

thanks! this is exactly what I was hoping for (community help) :) Esteban > On 25 May 2015, at 23:59, Nicolai Hess <nicolaihess(a)web.de> wrote: > > > > 2015-05-25 13:23 GMT+02:00 Nicolai Hess <nicolaihess(a)web.de <mailto:nicolaihess@web.de>>: > > > 2015-05-24 10:02 GMT+02:00 Esteban Lorenzano <estebanlm(a)gmail.com <mailto:estebanlm@gmail.com>>: > > > > - there are some issues remaining: > -- cells escaping his width > > > Morphic-Widgets-FastTable-NicolaiHess.7 > calculate column width (needed if all are undefined / see FTExamples exampleTable1) > > > > Name: Morphic-Widgets-FastTable-NicolaiHess.8 > better calculation of visible number of rows for scrolling, not perfect. > > >

May 26, 2015