Re: [Pharo-dev] New User and the Rectangle class.

2017-06-24 Thread Stephane Ducasse
We spent several days fixing the complete system to avoid this crappy broken rectangle. Now it is time to deprecated this bad api. https://pharo.fogbugz.com/f/cases/20180/Deprecate-fractions-offsets-and-fractions On Sat, Jun 24, 2017 at 9:42 PM, Stephane Ducasse wrote: > Here is the comment > >

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-24 Thread Stephane Ducasse
Here is the comment fractions: fractionsOrNil "Do not use this API if you do not have already have the rectangles that should be passed as argument. If you are creating the rectangles representing the numbers you need, better use the accessors: (LayoutFrame identity topOffset: topHeight; bottomFra

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-24 Thread Stephane Ducasse
Yes I thought that they were because they are a BAD design. No need to create dummy rectangle when you have a fully nice api. On Thu, Jun 22, 2017 at 3:48 PM, Trey Tomes wrote: > I did not know about the GitHub tutorial. I'll take a look for it. > > Will fraction:offset: be deprecated then you t

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-22 Thread Trey Tomes
I did not know about the GitHub tutorial. I'll take a look for it. Will fraction:offset: be deprecated then you think? ~Trey On Thu, Jun 22, 2017 at 12:20 AM, Stephane Ducasse wrote: > Avoid fraction:offset: it is a bad interface. > > > > On Thu, Jun 22, 2017 at 7:20 AM, Stephane Ducasse > w

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-21 Thread Stephane Ducasse
Avoid fraction:offset: it is a bad interface. On Thu, Jun 22, 2017 at 7:20 AM, Stephane Ducasse wrote: > Welcome Trey > > This is nice. Did you see that we started long time ago to have a > version of the original tutorial on github to port it to Pharo. Feel > free to send pull requests. > > Ab

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-21 Thread Stephane Ducasse
Welcome Trey This is nice. Did you see that we started long time ago to have a version of the original tutorial on github to port it to Pharo. Feel free to send pull requests. About this API. the old API was not good because it forced you to create a bogus rectangle often for nothing and also poi

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-21 Thread Ben Coman
On Wed, Jun 21, 2017 at 10:17 PM, Trey Tomes wrote: > Chris: Thank you for the reference to the topLeftOffset:bottomRightOffset: > API, that resolved my problem really well. > @all, Would this be suitable for a QA rule? cheers -ben > I'm glad to have a solution that doesn't have me modifying b

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-21 Thread Trey Tomes
Chris/Peter: Thank you for taking the time to respond. It's good to know that I'm not the only one to encounter this problem. Chris: Thank you for the reference to the topLeftOffset:bottomRightOffset: API, that resolved my problem really well. I'm glad to have a solution that doesn't have me mod

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-21 Thread Peter Uhnak
iirc there was proposal to rename Rectangle to DisplayRectangle (or something like that), because you are certainly not the first one to run into such trouble. P On Tue, Jun 20, 2017 at 04:03:48PM -0700, Chris Cunningham wrote: > Hi Trey, > I can't find the rational written down why Rectangle

Re: [Pharo-dev] New User and the Rectangle class.

2017-06-20 Thread Chris Cunningham
Hi Trey, I can't find the rational written down why Rectangle was changed (although it WAS written down). If I remember right, the rational was roughly "Rectangles represent areas of the screen. As such, they should be 'well formed'- positive width and height only". In any case, going back throug

[Pharo-dev] New User and the Rectangle class.

2017-06-20 Thread Trey Tomes
Hello! I'm new to this list, so I apologize if this topic has already been considered. I have been re-creating the Laser Game tutorial in Pharo 6, and have encountered something that may be an error within Pharo. The LayoutFrame has the ability to use negative values for the bottom and right sid