- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] transformmorph and submorph bounds

by Thierry Goubier

2014-07-23 11:10 GMT+02:00 Tudor Girba <tudor(a)tudorgirba.com>: > Hi, > > > > > >> This seems correct to me: a transformMorph is the one which shift its >> contents (and reset the origin to its top left corner ?). >> > > Not really. The contract of submorphBounds should be that it returns an > absolute coordinate, as far as I understand. > In owner coordinates it says. (TransformMorph) However, it effectively returns a 0@0 origin. > > >> Why do you use a TransformMorph there? >> > > It is used for example in ScrollPane. This means that if you embed a > TextMorph inside a ScrollPane, you will have the same problem with > completion: > > scrollPane := ScrollPane new. > pasteUpMorph := PasteUpMorph new. > scrollPane scroller addMorph: pasteUpMorph. > scrollPane openInWindow top: 100; left: 100. > > pasteUpMorph submorphBounds "(0@0) corner: (50@40)" > Yes. What do you want to do about it? Rewrite a scroll pane in a different way? Or make sure that everything which creates over-morphs ask for global coordinates? I guess this has also something to do with the glamour bug inside tabs, no? Thierry

July 23, 2014

Re: [Pharo-dev] SmartBreakpoints alpha release

by stepharo

On 22/7/14 14:43, Tudor Girba wrote: > First of all: Excellent job! > > About the editor: I would indeed suggest adopting Rubric as a solution > for now (until the new Text widget comes around). We use it > extensively in GT and I can attest that it is better than > PluggableTextMorph. doru did you loaded TxText? and TxViewContainer example2 > > Doru > > > On Tue, Jul 22, 2014 at 2:05 PM, Marcus Denker <marcus.denker(a)inria.fr > <mailto:marcus.denker@inria.fr>> wrote: > > > On 22 Jul 2014, at 13:57, stepharo <stepharo(a)free.fr > <mailto:stepharo@free.fr>> wrote: > >> >> On 22/7/14 10:41, Clara Allende wrote: >>> Hi guys! >>> Finally the metaLink-based breakpoints are "ready". I have made >>> some explanations on my blog >>> <https://clariallende.wordpress.com/2014/07/21/smartbreakpoints-alpha-releas…>, >>> but basically now you have the option "break here" in >>> SmartSuggestions menu. >>> So it's still experimental, and even though I have some tests, >>> it would be nice if you can test it, because I can't cover all >>> possible use cases :) >>> And any feedback is most appreciated! >>> >>> Specially, since eventually we will have several kinds of >>> breakpoints, I would like to ask in which way would you like to >>> choose them (a pop menu? a sublist?) >>> >>> Thanks in advance! >>> >>> PS: The script for downloading the code is in the blog post :P >>> PS2: After talking with Anne and Baptiste, things that are not >>> there *yet* but I want to have: >>> * a highlight on the method/messageSend where the breakpoint is >>> inserted >>> * expose persistent breakpoints (it is there but the default is >>> to halt once) >>> * line breakpoints (not sure when, for now the breakpoints are >>> AST based) >>> * integration with TextEditor, once it is done ;) >> >> Can you grab igor and sit with him to get this point? > Another idea would be to use the editor in Moose (Rubrik?). > Better a solution now than to wait forever for perfectionâ¦ > > Marcus > > > > > -- > www.tudorgirba.com <http://www.tudorgirba.com> > > "Every thing has its own flow"

July 23, 2014

Re: [Pharo-dev] SmartBreakpoints alpha release

by stepharo

>> Can you grab igor and sit with him to get this point? > Another idea would be to use the editor in Moose (Rubrik?). > Better a solution now than to wait forever for perfectionâ¦ We can also abandon TxText but after we should not cry that it does not scale (be prepared to fork a lot of processes in the background). But I think that it would be stupid so we should push igor to deliver. And if I'm alone pushing then this is not fun. Have a look at TxViewContainer example2 Stef > > Marcus >

July 23, 2014

Re: [Pharo-dev] First attempt at submitting a simple contribution

by stepharo

Thanks tim. Usually the monkey will tag your entry to ready to integrate and we harvest them. After this is also nice to write a comment saying that your change is ok in the latest image and we will integrate. We would like to change the integration process but we are not yet there. Git is needed first. We are getting more integrator: clement (on the compiler domain), nicolai and this is good. Sven is integrating changes related to the network Stef On 22/7/14 17:48, Tim Mackinnon wrote: > Hi guys - Iâve tried to submit a very simple contribution for Pharo. I followed the steps in the video about submitting on Fogbugz - although I wonder if I missed a step by not asking for commit rights to the Pharo40Inbox? As I already had a smalltalk hub account - it asked for that name and password and it didnât complain when I entered them and pressed accept in Pharo. > > The issue is: 13653 > > Its a very simple contribution, just a class with some wiki help and the wiki method pragma to get it to appear in the System help. > > As I didnât know if there was already a class that might better hold this text, I created a new one - but happy to move it onto something else if there is somewhere obvious (or maybe its cleaner that it lives in its own place?). > > Tim >

July 23, 2014

Re: [Pharo-dev] future of Nautilus (the non package tree version)

by stepharo

Yes but I think that it is necessary. What esteban introduced was nice: a design with less conditional. Stef > > On 22 Jul 2014, at 14:26, Nicolai Hess <nicolaihess(a)web.de > <mailto:nicolaihess@web.de>> wrote: > >> From time to time it happens that we introduce bugs in the "old" >> Nautilus >> browser if we do changes for the default browser >> (PackageTreeNautilus) >> For example: >> 13640 <https://pharo.fogbugz.com/default.asp?13640> >> no menu on package pane in Nautilus (the non package tree nautilus) >> >> Is a working "old" Nautilus on high priority? Or should we consider >> PackageTreeNautilus as a replacement for Nautilus? > > yes. > I made the PackageTreeNautilus as a replacement, but an extension > because I was using Nautilus to develop Nautilus. > Now we should merge it. > > > This might be a big effort, but this also gives us the opportunity to > clean up. > > > > Esteban > >> >> nicolai >> >> > >

July 23, 2014

Re: [Pharo-dev] Doubt in Pharo

by stepharo

Hi Dokania I hope that you do not doubt that Pharo is a cool system :) > Hello, > > I am creating an Object Relational Mapper in pharo using sqlite. > > While creating database I am using a dictionary for columns of Database. This gives me Keys as column name and values as Data type of those. > > So suppose Employee is the the table name, with ID->Integer, Name->Text, Salary->Float. > > emp:= Employee new. > Now emp is a row of table, so for each row user has to create instance of Employee and then feed values to it. Later on he can pass message insert query to emp. > > So I was creating this dictionary for column at the instance side initialization of Employee. > > My instructor suggested to do that in class side initialization, > > My Doubts are: > > 1. Why should we do class side initialization instead of instance side initialization. Because class side initialization is executing once when the class is loaded in memory. And I imagine that you do not want to change and execute the mapping declaration for every single instance you will create. isn't? So Object subclass: #Employee instanceVariableNames: '' classVariableNames: 'Mappings' category: 'Foo-Model' Employee class>>initialize Mappings := { #ID->Integer . #Name->Text. #Salary->Float } asDictionary makes a lot of sense. or Object subclass: #Employee instanceVariableNames: '' classVariableNames: '' category: 'Foo-Model' Employee class instanceVariableNames: 'mappings' Employee class>>initialize mappings := { #ID->Integer . #Name->Text. #Salary->Float } asDictionary Read pharo by example to understand the difference between class variable (Shared Variable) and class instance variables > > 2. Could you please refer me to the link which explains how to do class side initialization. Because when I do Employee new, I see only instance side initialization is being called. Employee initialize you send a message to the CLASS itself to get **ITS** initialize method execute. Normally when you load the class its initialize method is automatically executed. When you do Employee new => Employee basicNew initialize => anEmployee initialize => anInitializedEmployee > > Thanks, > Harshit > > >

July 23, 2014

Re: [Pharo-dev] transformmorph and submorph bounds

by Tudor Girba

Hi, On Wed, Jul 23, 2014 at 9:52 AM, Thierry Goubier <thierry.goubier(a)gmail.com> wrote: > > > > 2014-07-23 8:00 GMT+02:00 Tudor Girba <tudor(a)tudorgirba.com>: > > Hi, >> >> In GTInspector we use a TransformMorph inside the pager morph. It works >> fine, but spawning completion from a text morph that is placed inside this >> transform morph, the completion morphs appears in the wrong place. >> >> The issue comes from submorphBounds apparently not being right (this is >> what the completion morph relies on). >> >> Here is a simpler example, to exemplify what I am talking about: >> >> circle := CircleMorph new. >> morph := TransformMorph new. >> morph addMorph: circle. >> morph top: 100. >> morph left: 100. >> morph submorphBounds. "==> (100@100) corner: (140@140)" >> circle submorphBounds. "==> (0@0) corner: (40@40)" >> > > This seems correct to me: a transformMorph is the one which shift its > contents (and reset the origin to its top left corner ?). > Not really. The contract of submorphBounds should be that it returns an absolute coordinate, as far as I understand. > Why do you use a TransformMorph there? > It is used for example in ScrollPane. This means that if you embed a TextMorph inside a ScrollPane, you will have the same problem with completion: scrollPane := ScrollPane new. pasteUpMorph := PasteUpMorph new. scrollPane scroller addMorph: pasteUpMorph. scrollPane openInWindow top: 100; left: 100. pasteUpMorph submorphBounds "(0@0) corner: (50@40)" Cheers, Doru > Thierry > > >> Ideally, "circle submorphBounds" should start at 100@100. Is this a bug, >> or am I doing something wrong? >> >> Cheers, >> Doru >> >> -- >> www.tudorgirba.com >> >> "Every thing has its own flow" >> > > -- www.tudorgirba.com "Every thing has its own flow"

July 23, 2014

[pharo-project/pharo-core] 0c052f: 40117

by GitHub

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: 0c052fbe84db7ec2db77bdac9ea4d8448638c2c4 https://github.com/pharo-project/pharo-core/commit/0c052fbe84db7ec2db77bdac… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-07-23 (Wed, 23 Jul 2014) Changed paths: R Kernel.package/CompiledMethod.class/instance/accessing/searchForClass.st R Kernel.package/CompiledMethod.class/instance/accessing/searchForSelector.st R KernelTests.package/CompiledMethodTest.class/instance/tests - accessing/testSearchForClass.st R KernelTests.package/CompiledMethodTest.class/instance/tests - accessing/testSearchForSelector.st M Refactoring-Critics.package/RBConsistencyCheckRule.class/instance/accessing/name.st M Refactoring-Critics.package/RBConsistencyCheckRule.class/instance/initialization/initialize.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script117.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40117.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 40117 13656 remove CompiledMethod searchForClass and searchForSelector https://pharo.fogbugz.com/f/cases/13656 13657 Remove at: check from RBConsistencyCheckRule https://pharo.fogbugz.com/f/cases/13657 http://files.pharo.org/image/40/40117.zip

July 23, 2014

[pharo-project/pharo-core]

by GitHub

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

July 23, 2014

Re: [Pharo-dev] we need you on http://pharo.org/about

by Ben Coman

July 23, 2014