Issue 6040 in pharo: menu addToggle: - does not create checkbox in a menu

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 6040 in pharo: menu addToggle: - does not create checkbox in a menu

pharo
Status: New
Owner: ----
Labels: Milestone-1.4 Type-Bug

New issue 6040 by [hidden email]: menu addToggle: - does not create  
checkbox in a menu
http://code.google.com/p/pharo/issues/detail?id=6040

The problem is that hasIcon is the method of the MenuItemMorph. So the  
ToggleMenuIemMorph appears without the checkbox even when has correct state  
selector.

The workaround is simple, after the addToggle: just run -
"menu lastItem icon: menu lastItem onIcon."

The proposed changes - add the method hasIcon into the ToggleMenuItemMorph  
itself. Like this -

hasIcon
self getStateSelector ifNil: [^super hasIcon].
^true


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6040 in pharo: menu addToggle: - does not create checkbox in a menu

pharo
Updates:
        Status: HumanReviewNeeded

Comment #1 on issue 6040 by [hidden email]: menu addToggle: - does  
not create checkbox in a menu
http://code.google.com/p/pharo/issues/detail?id=6040

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6040 in pharo: menu addToggle: - does not create checkbox in a menu

pharo
Updates:
        Status: WorkNeeded

Comment #2 on issue 6040 by [hidden email]: menu addToggle: -  
does not create checkbox in a menu
http://code.google.com/p/pharo/issues/detail?id=6040

mbalkarov,

do you have a snippet of code to reproduce the problem?


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker