Issue 1036 in moose-technology: Ctrl + S and Alt + S haven't the same behaviour

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 1036 by [hidden email]: Ctrl + S and Alt + S haven't the  
same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

Describe the problem: what do you get? what do you expect?

To save the modified source code in the debugger under Pharo3.0, we can  
make Ctrl + S or Alt + S.

But with the latest version of Moose5.0. Alt + S save the source and move  
the stack to the saved method. Ctrl + S only save the source.

How to reproduce the problem: step by step if necessary

Open a debugger, select an other method in the stack trace, modify it.
Try to save with Ctrl + S and do it again with Alt + S.

Additional information: platform, context which may impact the problem

Windows 7 x64 and Linux Ubuntu x64

Please fill in the labels with the following information:
* Type-Defect
* Component-GTDebugger
* Milestone-5.0

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #1 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

This Ctrl+s problem is so annoying and it's a morph problem. I do not have  
a Windows machine nearby, but could you try to see if Ctrl+s/Alt+s work in  
the code browser as expected?

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Updates:
        Labels: Platform-Pharo OpSys-Linux

Comment #2 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

I have no problems in code browser.

But I have the problem in debugger:
Alt+s saves the code as expected.
Ctrl+s looks like it saved the code (it changes in the debugger) but the  
change is not visible in browser and next time the debugger pops up the  
change is not visible.

This makes me crazy :)

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Updates:
        Labels: -Platform-Pharo -OpSys-Linux

Comment #3 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #4 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

I have no problems in code browser.

But I have the problem in debugger:
Alt+s saves the code as expected.
Ctrl+s looks like it saved the code (it changes in the debugger) but the  
change is not visible in browser and next time the debugger pops up the  
change is not visible.

This makes me crazy :)

I use Linux Ubuntu x64

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Updates:
        Labels: Component-Glamour Milestone-5.0

Comment #5 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

I hate this mess with text morph. Ctrl+s comes from the low level text  
morph. This binding should not exist actually, and everything would be fine.

I have to check on Windows, too.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #6 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

I lost some lines of code because of this bug....

I suggest to overload GLMTextMorphForEditView>>acceptContents with an empty  
method.
By this way, the code will not seem to be saved.

But if we close the window containing modified code, no message is  
displayed.
Can we display such a message ?

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #7 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

I committed the change I suggested.
It's in Glamour-Morphic-Widgets version 77.

It's a temporary hack only...

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #8 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

Thanks!

I will test it tomorrow on a Windows machine. Can anyone else take a look  
as well on Windows and Linux?

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Updates:
        Status: Fixed

Comment #9 on issue 1036 by [hidden email]: Ctrl + S and Alt + S  
haven't the same behaviour
http://code.google.com/p/moose-technology/issues/detail?id=1036

It works fine. Ctrl+s is disabled on Windows.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev