Issue 3514 in pharo: Failing Test: NewTextMorphTests.testAcceptContents

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

Issue 3514 in pharo: Failing Test: NewTextMorphTests.testAcceptContents

pharo
Status: Accepted
Owner: marcus.denker
Labels: Milestone-1.2

New issue 3514 by marcus.denker: Failing Test:  
NewTextMorphTests.testAcceptContents
http://code.google.com/p/pharo/issues/detail?id=3514

NewTextMorphTests.testAcceptContents fails


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3514 in pharo: Failing Test: NewTextMorphTests.testAcceptContents

pharo

Comment #1 on issue 3514 by guillermopolito: Failing Test:  
NewTextMorphTests.testAcceptContents
http://code.google.com/p/pharo/issues/detail?id=3514

This is because TextEditor>>accept does:

accept
        "Save the current text of the text being edited as the current acceptable  
version for purposes of canceling.  Allow my morph to take appropriate  
action"
        morph enableTextChange
                ifTrue: [morph acceptContents]


and morph (a NewTextMorph) dnu #enableTextChange.
Does NewTextMorph has the capability of beign enabled or disabled?  Shoud  
it do only

morph acceptContents

?




Reply | Threaded
Open this post in threaded view
|

Re: Issue 3514 in pharo: Failing Test: NewTextMorphTests.testAcceptContents

pharo
Updates:
        Status: Fixed

Comment #2 on issue 3514 by aplantec: Failing Test:  
NewTextMorphTests.testAcceptContents
http://code.google.com/p/pharo/issues/detail?id=3514

just make EditableTextMorph understand #enableTextChange

EditableTextMorph>>enableTextChange
        ^ self enabled

Fixed
SLICE-Issue-3514-Failing-Test-NewTextMorphTeststestAcceptContents


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3514 in pharo: Failing Test: NewTextMorphTests.testAcceptContents

pharo

Comment #3 on issue 3514 by stephane.ducasse: Failing Test:  
NewTextMorphTests.testAcceptContents
http://code.google.com/p/pharo/issues/detail?id=3514

Thanks.
I need to relax from proposal writing and I will do some integration  
after.... (but I need to read a phd).


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3514 in pharo: Failing Test: NewTextMorphTests.testAcceptContents

pharo
Updates:
        Status: Closed

Comment #4 on issue 3514 by marcus.denker: Failing Test:  
NewTextMorphTests.testAcceptContents
http://code.google.com/p/pharo/issues/detail?id=3514

12295