Issue 6603 in pharo: KeyMapping - <ctrl-a> produces MNU PluggableTextMorph>>cursorHome

Status: New
Owner: ----
Labels: Type-Bug

New issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

In all text editors, and as a specific example, in Workspace, normally I  
press <ctrl-a> to "Select All" text, but currently the is producing a MNU  
PluggableTextMorph>>cursorHome.

The problem occurs with Pharo-1.4-14557, but not Pharo-1.4-14338 or  
Pharo-2.0-build-2012-08-23,
all in Windows 7.  This appears related to the introduction of Keymapping  
in 14557.

By logging to the Transcript from HandMorph>>processEvents, I determined  
that the related evtBuf was #(2 XXXXXX 97 0 8 1 0 1).  With the following  
code to both 14557 & 14338...
     evtBufDebugCopy := evtBuf copy at: 2 put: 0; yourself.
     (evtBufDebugCopy = #(2 0 97 0 8 1 0 1)) ifTrue: [Transcript  
crShow: 'BANG'. Halt now].

...I traced the execution path of both 14557 & 14338 and found them to be  
the same.  That is, both versions generateKeyboardEvent: return a  
keystroke '<Cmd-home>'.

That is as far I can get.  In case this problem is Windows specific, for  
other platforms to reproduce the problem, I was able to simulate it with  
the following in HandMorph>>processEvents...
     [(evtBuf := Sensor nextEvent) isNil] whileFalse:
         [
             (Smalltalk at: #simulateEvtBuf ifAbsent: [] ) ifNotNil:
             [ :simEvtBuf |
                 Smalltalk at: #simulateEvtBuf put: nil.
                 Transcript crShow: simEvtBuf.
                 evtBuf := simEvtBuf
             ].

and then from workspace....
     Smalltalk at: #simulateEvtBuf put: #(2 0 97 0 8 1 0 1)



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

Comment #1 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

Same as Issue 6555


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

Updates:
        Status: Accepted
        Labels: Milestone-1.4 Milestone-2.0

Comment #2 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

(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] [hidden email]

Comment #3 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

This fixes the DNU for home/end, but the question remains as to why  
cursorHome: was ever sent with ctrl-a in the first place. This should only  
be sent with ctrl-"up arrow". I could reproduce the DNU on the Mac, but not  
with ctrl-a or cmd-a. See if you get the same error in 14459...

Fix in inbox:  
SLICE-Issue-6603-KeyMapping---ltctrl-agt-produces-MNU-PluggableTextMorphgtgtcursorHome-SeanDeNigris.1

- Forward cursorHome:/cursorEnd: from PluggableTextMorph to its editor


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

Updates:
        Status: MonkeyIsChecking

Comment #4 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c4

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: CheckedIn20507

Comment #5 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c5

This Issue has been checked by Ulysse the Monkey
       
6433 tests passed in 00:01:27s:
===============================
        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 (69)
        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)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-6603-KeyMapping---ltctrl-agt-produces-MNU-PluggableTextMorphgtgtcursorHome-SeanDeNigris.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20507-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: MonkeyIsChecking

Comment #6 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c6

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: CheckedIn20509

Comment #7 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c7

This Issue has been checked by Ulysse the Monkey
       
6433 tests passed in 00:01: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 (69)
        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)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-6603-KeyMapping---ltctrl-agt-produces-MNU-PluggableTextMorphgtgtcursorHome-SeanDeNigris.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20509-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

Comment #8 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

> but the question remains as to why cursorHome: was ever sent with ctrl-a  
> in the first place. This should only be sent with ctrl-"up arrow".

Perhaps related...  
http://en.wikipedia.org/wiki/Table_of_keyboard_shortcuts#Text_editing
"Go to start of line"


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

Comment #9 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

@ben: Ah, yes that looks like it... but why did it all of a sudden start  
doing that when it used to "select all"


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

Updates:
        Labels: -Milestone-1.4

Comment #10 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

I don't think we're doing any more fixes for 1.4, so I'll remove the label.


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

Comment #11 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

Okay. For myself in 1.4, after a while I adapted to using Alt-A. However I  
still get caught-out when I am pasting between Pharo and another  
application and needing to switch between Ctrl & Alt.

Ctrl-A works for Select-All in 20510.  However I notice that Ctrl-Home also  
does a Select-All when I would expect it to go to the start of the text.  
That should not hold up 2.0 but would be good if it can be looked at later.


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

Comment #12 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

@ben: I opened Issue 7415 tagged 3.0 to look at the shortcut conflict.


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

Updates:
        Status: MonkeyIsChecking

Comment #13 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c13

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: CheckedIn20510

Comment #14 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c14

This Issue has been checked by Ulysse the Monkey
       
6433 tests passed in 00:01:29s:
===============================
        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 (69)
        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)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-6603-KeyMapping---ltctrl-agt-produces-MNU-PluggableTextMorphgtgtcursorHome-SeanDeNigris.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20510-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: MonkeyIsChecking

Comment #15 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c15

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: CheckedIn20511

Comment #16 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603#c16

This Issue has been checked by Ulysse the Monkey
       
6433 tests passed in 00:01:19s:
===============================
        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 (69)
        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)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-6603-KeyMapping---ltctrl-agt-produces-MNU-PluggableTextMorphgtgtcursorHome-SeanDeNigris.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20511-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: HumanReviewNeeded

Comment #17 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

(No comment was entered for this change.)


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

Comment #18 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

For the human to test ;)...

1. In a Nautilus source code pane, press (on Macbook) "Cmd-fn-[left arrow]"
2. You will see a DNU
3. Repeat for right arrow
4. Another DNU
5. Load slice
6. Repeat steps 1 and 3... should be no DNUs



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

Updates:
        Status: FixToInclude

Comment #19 on issue 6603 by [hidden email]: KeyMapping - <ctrl-a>  
produces MNU PluggableTextMorph>>cursorHome
http://code.google.com/p/pharo/issues/detail?id=6603

It's working :)


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