[Pharo-project] Issue 3996 in pharo: alt+k opens a StringHolder and not a workspace
Status: Started Owner: damien.c...@gmail.com Labels: Type-ReportDefect Milestone-1.2.2-DevImage Difficulty-Easy New issue 3996 by damien.c...@gmail.com: alt+k opens a StringHolder and not a workspace http://code.google.com/p/pharo/issues/detail?id=3996 alt+k shortcut is defined in PasteUpMorph>>defaultDesktopCommandKeyTriplets { $k. StringHolder. #open. 'Open a new, blank Workspace'}. Then, alt+k opens a StringHolder and not a plain Workspace. This prevents the user from accessing all features of workspaces (syntax highlighting, old content management, line ending handling...). It would be better to open a workspace. The correct way to do that is probably to ask the default ToolSet: ToolSet default openWorkspace However, the syntax of the table in #defaultDesktopCommandKeyTriplets prevents sending 2 messages to ToolSet. Doing { $k. ToolSet default. #openWorkspace. 'Open a new, blank Workspace'}. is fragile because it fixes the toolset upon initialization of the World. I propose to add #openWorkspace to the ToolSet class (along with a lot of other useful utilities) which would look like: ToolSet class>>openWorkspace ^ self default openWorkspace and then update #defaultDesktopCommandKeyTriplets: { $k. ToolSet. #openWorkspace. 'Open a new, blank Workspace'}.
Updates: Status: FixProposed Comment #1 on issue 3996 by damien.c...@gmail.com: alt+k opens a StringHolder and not a workspace http://code.google.com/p/pharo/issues/detail?id=3996 Name: SLICE-Issue-3996-altk-opens-a-StringHolder-and-not-a-workspace-DamienCassou.1 Author: DamienCassou Time: 12 April 2011, 11:03:15 am UUID: 7f3fb192-6dab-405b-8a5f-00a8701fe323 Ancestors: - Updates PasteUpMorph>>defaultDesktopCommandKeyTriplets so that it uses the ToolSet to open a workspace - Adds ToolSet class>>openWorkspace
Comment #2 on issue 3996 by damien.c...@gmail.com: alt+k opens a StringHolder and not a workspace http://code.google.com/p/pharo/issues/detail?id=3996 Use version 2 instead, as version 1 was an empty slice. Also requires to execute World initializeDesktopCommandKeySelectors to update the key strokes.
participants (1)
-
pharo@googlecode.com