Status: New Owner: ---- Labels: Type-Defect Priority-Medium
New issue 1036 by v.blonde...@gmail.com: 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
Comment #1 on issue 1036 by tu...@tudorgirba.com: 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?
Updates: Labels: Platform-Pharo OpSys-Linux
Comment #2 on issue 1036 by kurs....@gmail.com: 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 :)
Updates: Labels: -Platform-Pharo -OpSys-Linux
Comment #3 on issue 1036 by kurs....@gmail.com: 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.)
Comment #4 on issue 1036 by kurs....@gmail.com: 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
Updates: Labels: Component-Glamour Milestone-5.0
Comment #5 on issue 1036 by tu...@tudorgirba.com: 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.
Comment #6 on issue 1036 by v.blonde...@gmail.com: 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 ?
Comment #7 on issue 1036 by v.blonde...@gmail.com: 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...
Comment #8 on issue 1036 by tu...@tudorgirba.com: 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?
Updates: Status: Fixed
Comment #9 on issue 1036 by tu...@tudorgirba.com: 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.