Issue 5220 in pharo: MenuRegistration do not allow menu with more than 2 levels of deapth

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

Issue 5220 in pharo: MenuRegistration do not allow menu with more than 2 levels of deapth

pharo
Status: FixReviewNeeded
Owner: [hidden email]
CC: [hidden email]
Labels: Type-Bug Milestone-1.4

New issue 5220 by [hidden email]: MenuRegistration do not allow menu  
with more than 2 levels of deapth
http://code.google.com/p/pharo/issues/detail?id=5220

MenuRegistration>>#itemNamed: aName
        "Return the MenuRegistration named aName or nil if none is found"
        self name = aName ifTrue: [^ self].
        ^ self itemList ifNotNil: [:l | l detect: [:item | (item itemNamed: aName)  
notNil] ifNone: []]

Here after few draws and some tests, you figure out that you return the  
first element whose itemList +contains+ the item named aName.

So I fix it to return the item named aName.

Attachments:
        MenuRegistrationAllowMultipleLevels.1.cs  684 bytes


_______________________________________________
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 5220 in pharo: MenuRegistration do not allow menu with more than 2 levels of deapth

pharo
Updates:
        Status: FixToInclude

Comment #1 on issue 5220 by [hidden email]: MenuRegistration do not  
allow menu with more than 2 levels of deapth
http://code.google.com/p/pharo/issues/detail?id=5220

(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 5220 in pharo: MenuRegistration do not allow menu with more than 2 levels of deapth

pharo
Updates:
        Status: Integrated

Comment #2 on issue 5220 by [hidden email]: MenuRegistration do not  
allow menu with more than 2 levels of deapth
http://code.google.com/p/pharo/issues/detail?id=5220

in 14298


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