And another... That'll have to do for now. The tab focus things will take quite some investigation... Regards, Gary ----- Original Message ----- From: "Gary Chambers" <gazzaguru2@btinternet.com> To: <Pharo-project@lists.gforge.inria.fr> Sent: Wednesday, April 25, 2012 3:52 PM Subject: Re: [Pharo-project] Polymorph (and a few other) changes/fixes
Fixes for checkbox, Benjamin, please check as I had to revert the CheckboxMorph>>initialize to not hook the click event.
Also I notice that tabbing from lists (and shift-tabbing from multiline text) appears to have been broken at some point before these changes.
Attached lastest cs
Regards, Gary
----- Original Message ----- From: "Gary Chambers" <gazzaguru2@btinternet.com> To: <Pharo-project@lists.gforge.inria.fr> Sent: Wednesday, April 25, 2012 3:36 PM Subject: Re: [Pharo-project] Polymorph (and a few other) changes/fixes
Indeed.
Hold off on integration for a few mins, some last minute fixes...
Regards, Gary
----- Original Message ----- From: "Stéphane Ducasse" <stephane.ducasse@inria.fr> To: <Pharo-project@lists.gforge.inria.fr> Sent: Wednesday, April 25, 2012 1:17 PM Subject: Re: [Pharo-project] Polymorph (and a few other) changes/fixes
In the future we will have to look at respondsTo: and remove them because we should enforce for example that objects that are used have the right protocol
|label| label := self checkboxLabelForText: aCheckbox label. (label respondsTo: #enabled:) ifTrue: [ label enabled: aCheckbox enabled]. label font: self labelFont. ^label
On Apr 25, 2012, at 1:53 PM, Stéphane Ducasse wrote:
I see such idiom and I'm thinking that it would be good to fix such kind of respondsTo:
self allMorphsDo: [:m | (m ~= self and: [m respondsTo: #font:]) ifTrue: [m font: aFont]]
Stef