2011/1/31 Stéphane Ducasse <[email protected]>: > The package should be updated to 1.2. > > It should declare fonts probably in one of the classVar of the TextConstant > SharedPool. > Probably > TextSharedInformation have a look. > > In 1.2 we cleaned the last global poolvar = textConstant which was used as a > pool but also as a repository to plug anything inside > instead of specific classVariable in adequate classes. It was a good plate of > spaghettis code. > > I know that benjamin wants to take some times to do another pass and move the > left over to the class they belongs to but it will > take some time. > > Stef >
Nice effort to clean this up. Among the so called "Constants", I remember browsing one recently : (TextConstants at: #pixelsPerInch). Uh, this one should be device dependent, shouldn't it ? No, 96 seems the only figures written in the stone, and you'll hardly find any other thing but this default value in a Squeak image. Moreover, if you analyze Squeak code, you'll see some hardcoded 96 DPI here and there. Hope Pharo will give a kick in the anthill (literal translation of a french saying). Nicolas
