Et voilà , I just opened the issue, the number given by fogbugz is 17951. Thanks again - Carlos On Thu, Apr 7, 2016 at 1:13 PM, Carlos Lombardi <carlombardi@gmail.com> wrote:
Hi Stephan,
thank you for your prompt answer.
We are actually using TextEditor>>changeTextFont, which initiates the following message call chain TextEditor>>changeTextFont StrikeFont class>>fromUser: MorphicUIManager>>fontFromUser: StrikeFont class>>fromUser:allowKeyboard:
Therefore, the method StrikeFont class>>fromUser: is already used. It is handy to us to use TextEditor>>changeTextFont, because it takes as default the font used in the currenty pointed character of the editor.
Effectively, removing the use of the translated method in the MenuMorph method you transcribed solves the problem. I will try to open an issue.
Cordially - Carlos Lombardi
On Wed, Apr 6, 2016 at 9:16 AM, Stephan Eggermont <stephan@stack.nl> wrote:
On 05-04-16 23:22, Carlos Lombardi wrote:
I provisionally patched this issue by modifying the class method in StrikeFont, in order to give the menu Strings instead of Associations. As far as this way of selecting a point size is still usable, it would be nice to have this issue fixed. If there is another, maybe more modern, way of graphically choosing a point size, please let me know.
This is fixed in Pharo 5, if I understand you correctly:
StrikeFont fromUser: StrikeFont fromUser
seems to do the right thing.
The translated was added 2015-4-25 and removed again 2015-9-3 Current implementation is
MenuMorph>addToggle: aString target: anObject selector: aSymbol getStateSelector: stateSymbol enablementSelector: enableSymbol argumentList: argList "Append a menu item with the given label. If the item is selected, it will send the given selector to the target object."
|item| item := ToggleMenuItemMorph new contents: aString; target: anObject; selector: aSymbol; arguments: argList; getStateSelector: stateSymbol; enablementSelector: enableSymbol. ^ self addMenuItem: item
If this solves the problem, you can open an issue for this and we can backPort it
Stephan