Issue 7115 in pharo: cmd . in workspace erase the line

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Milestone-1.4 Milestone-2.0

New issue 7115 by [hidden email]: cmd . in workspace erase the line
http://code.google.com/p/pharo/issues/detail?id=7115

I don't know if it's a bug but doing cmd + . in the workspace not only  
shows user interrupt window but also erases the lines selected replacing  
them by "."


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

Updates:
        Status: HumanReviewNeeded

Comment #1 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115

(No comment was entered for this change.)


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

Updates:
        Labels: Importance-High

Comment #2 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115

And doing cmd + . in some modal dialog (like the one you occasionaly get to  
enter your initials) just display the . and does not even raise a  
debugger...
(Pharo2.0a - Latest update: #20433)


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

Updates:
        Status: WorkNeeded

Comment #3 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115

(No comment was entered for this change.)


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

Updates:
        Status: FixReviewNeeded
        Cc: [hidden email]

Comment #4 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115

Fix in inbox:  
SLICE-Issue-7115-cmd--in-workspace-erase-the-line-SeanDeNigris.1
- Treat user interrupt events as navigation keys

I tried to go higher up and intercept in Morph>>keyStroke:, but it seems  
like #navigationKey: is the only hook to prevent keys from getting to text  
morphs...

@nicolas: As far as not raising the debugger, it seems to be a separate  
issue, and by design. The user interrupt is indeed handled, but if certain  
processes (e.g. the idle process) are active,  
UserInterruptHandler>>#processToInterrupt returns nil and no debugger is  
opened.

Maybe we could throw up a growl morph


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

Updates:
        Status: MonkeyIsChecking

Comment #5 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115#c5

The Monkey is currently checking this issue. Please don't change it!


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

Updates:
        Status: ValidatedByTheMonkey
        Labels: CheckedIn20498

Comment #6 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115#c6

This Issue has been checked by Ulysse the Monkey
       
6442 tests passed in 00:03:18s:
===============================
        CollectionsTests-Arrayed (553)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (912)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (611)
        CollectionsTests-Support (12)
        CollectionsTests-Unordered (1954)
        CollectionsTests-Weak (739)
        CompilerTests (181)
        KernelTests-Chronology (593)
        KernelTests-Classes (68)
        KernelTests-Exception (2)
        KernelTests-Methods (179)
        KernelTests-Numbers (276)
        KernelTests-Objects (86)
        KernelTests-Pragmas (3)
        KernelTests-Processes (38)
        MorphicTests-Basic (12)
        MorphicTests-Event (9)
        MorphicTests-Kernel (8)
        MorphicTests-Layouts (5)
        MorphicTests-Text Support (4)
        MorphicTests-Widgets (3)
        MorphicTests-WindowNotification (8)
        SUnit-Core-Extensions (3)
        SUnit-Core-Utilities (3)
        SUnit-Tests-Core (78)
        Tests-Polymorph-Widgets (10)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-7115-cmd--in-workspace-erase-the-line-SeanDeNigris.1 from  
http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20498-a on CoInterpreter  
VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Dec 12 2012
StackToRegisterMappingCogit VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Dec 12 2012
git://gitorious.org/cogvm/blessed.git Commit:  
452863bdfba2ba0b188e7b172e9bc597a2caa928 Date: 2012-12-07 16:49:46 +0100  
By: Esteban Lorenzano <[hidden email]> Jenkins build #5922


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

Updates:
        Status: MonkeyIsChecking

Comment #7 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115#c7

The Monkey is currently checking this issue. Please don't change it!


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

Updates:
        Status: ValidatedByTheMonkey
        Labels: CheckedIn20499

Comment #8 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115#c8

This Issue has been checked by Ulysse the Monkey
       
6442 tests passed in 00:03:14s:
===============================
        CollectionsTests-Arrayed (553)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (912)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (611)
        CollectionsTests-Support (12)
        CollectionsTests-Unordered (1954)
        CollectionsTests-Weak (739)
        CompilerTests (181)
        KernelTests-Chronology (593)
        KernelTests-Classes (68)
        KernelTests-Exception (2)
        KernelTests-Methods (179)
        KernelTests-Numbers (276)
        KernelTests-Objects (86)
        KernelTests-Pragmas (3)
        KernelTests-Processes (38)
        MorphicTests-Basic (12)
        MorphicTests-Event (9)
        MorphicTests-Kernel (8)
        MorphicTests-Layouts (5)
        MorphicTests-Text Support (4)
        MorphicTests-Widgets (3)
        MorphicTests-WindowNotification (8)
        SUnit-Core-Extensions (3)
        SUnit-Core-Utilities (3)
        SUnit-Tests-Core (78)
        Tests-Polymorph-Widgets (10)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-7115-cmd--in-workspace-erase-the-line-SeanDeNigris.1 from  
http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20499-a on CoInterpreter  
VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Dec 12 2012
StackToRegisterMappingCogit VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Dec 12 2012
git://gitorious.org/cogvm/blessed.git Commit:  
452863bdfba2ba0b188e7b172e9bc597a2caa928 Date: 2012-12-07 16:49:46 +0100  
By: Esteban Lorenzano <[hidden email]> Jenkins build #5922


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

Updates:
        Status: MonkeyIsChecking

Comment #9 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115#c9

The Monkey is currently checking this issue. Please don't change it!


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

Updates:
        Status: ValidatedByTheMonkey
        Labels: CheckedIn20500

Comment #10 on issue 7115 by [hidden email]: cmd . in workspace  
erase the line
http://code.google.com/p/pharo/issues/detail?id=7115#c10

This Issue has been checked by Ulysse the Monkey
       
6442 tests passed in 00:02:23s:
===============================
        CollectionsTests-Arrayed (553)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (912)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (611)
        CollectionsTests-Support (12)
        CollectionsTests-Unordered (1954)
        CollectionsTests-Weak (739)
        CompilerTests (181)
        KernelTests-Chronology (593)
        KernelTests-Classes (68)
        KernelTests-Exception (2)
        KernelTests-Methods (179)
        KernelTests-Numbers (276)
        KernelTests-Objects (86)
        KernelTests-Pragmas (3)
        KernelTests-Processes (38)
        MorphicTests-Basic (12)
        MorphicTests-Event (9)
        MorphicTests-Kernel (8)
        MorphicTests-Layouts (5)
        MorphicTests-Text Support (4)
        MorphicTests-Widgets (3)
        MorphicTests-WindowNotification (8)
        SUnit-Core-Extensions (3)
        SUnit-Core-Utilities (3)
        SUnit-Tests-Core (78)
        Tests-Polymorph-Widgets (10)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-7115-cmd--in-workspace-erase-the-line-SeanDeNigris.1 from  
http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20500-a on NBCoInterpreter  
NativeBoost-CogPlugin-IgorStasenko.15 uuid:  
44b6b681-38f1-4a9e-b6ee-8769b499576a Nov 27 2012
NBCogit NativeBoost-CogPlugin-IgorStasenko.15 uuid:  
44b6b681-38f1-4a9e-b6ee-8769b499576a Nov 27 2012
https://git.gitorious.org/cogvm/blessed.git Commit:  
40ac7e7bdec6fef0e934d2c019b86db996053912 Date: 2012-11-19 18:54:49 +0100  
By: Mariano Martinez Peck <[hidden email]> Jenkins build #146


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

Updates:
        Status: FixToInclude

Comment #11 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115

(No comment was entered for this change.)


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

Updates:
        Status: Integrated

Comment #12 on issue 7115 by [hidden email]: cmd . in workspace erase  
the line
http://code.google.com/p/pharo/issues/detail?id=7115

in 20503


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

Comment #13 on issue 7115 by [hidden email]: cmd . in workspace  
erase the line
http://code.google.com/p/pharo/issues/detail?id=7115

"Maybe we could throw up a growl morph"
->
Issue 7477: [ENH]: User Interrupt: Inform user if interrupt not possible


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