[Moose-dev] Re: Mondrian-Alexandre_Bergel.302

The comment of this version says: -=-=-=-=-=-=-=-=-=-=-=-= This version contains a number of important refactorings and improvement: - MOGraphElement.children has been renamed into MOGraphElement.nodes - allChildrenDo: -> allNodesDo: - added a method MOGraphElement>>handleException: that handle error. Normally, an image should not freeze when rendering an erroneous Mondrian script. - Bug fixed in MOGraphElement>>elementAt:ifAbsent: This solve the ISSUE #193. A test has been added: MOViewRendererTest>>testEdgeContainsPointWithEdges - recagorization of methods in MOArrowedOrthoVerticalLineShape - Bug fixed in MOLineShape>>containsPoint:for: This solved the ISSUE #249. The problem stems from using CurvedLine. I added a test MOViewRendererTest>>testEdgeContainsPointCurveFitter for this - Added a safeguard in MOAnnouncer>>popupText: All 164 Test methods are green! -=-=-=-=-=-=-=-=-=-=-=-= Doru, I wanted to do a merge with tg.296, but you redefined MOSplineShape>>display:on: and I provide a new version of this method (I refactored some part). Maybe you want to do the merging of this method. I can do it if you want. I am now updating the corresponding issues in code.google.com Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list Moose-dev(a)iam.unibe.ch https://www.iam.unibe.ch/mailman/listinfo/moose-dev