Disable Esc (exit full screen mode) accelerator in Terminal.

[wadeb]

It's often handy to use Terminal in full screen mode.

Some terminal programs, like VIM, require use of the Esc key, but the Esc key exits full screen mode.

Terminal should disable the Esc accelerator, like it does with other accelerators.

[wadeb]

Not related to a milestone, just an enhancement. There is code in there to do it, but it's not working correctly.

[humitos]

I was taking a look at this problem. I found something strange here.

I put a log.debug(self) and log.debug(self.__key_press_cb) in sugar.graphics.window.Window.__init__ to know if self was TerminalActivity and that is True.

Besides I put a log.debug('TerminalActivity.__key_press_cb') inside the TerminalActivity.__key_press_cb method to know if it's called.

11:48 < humitos> 1332254619.978885 DEBUG Window: <TerminalActivity object at 0x8c84a2c (SugarActivity at 0x8bdd170)>
11:48 < humitos> 1332254619.979220 DEBUG Window: <bound method TerminalActivity.__key_press_cb of <TerminalActivity object at 0x8c84a2c (SugarActivity at 0x8bdd170)>>

So, I don't understand why TerminalActivity.__key_press_cb is never called. In fact, it's never connected by the Window class.

I commented the connection line in Window class and put that line (the connection one) inside the TerminalActivity class and it works properly.

[humitos]

Here are some information about this:

•  http://stackoverflow.com/questions/1301346/the-meaning-of-a-single-and-a-double-underscore-before-an-object-name-in-python

[humitos]

I found and fix this problem. I'm talking with "manuq" to know how to create a patch for this.

[humitos]

I've already sent the patch to the sugar-devel list:

•  http://lists.sugarlabs.org/archive/sugar-devel/2012-March/036230.html

[RafaelOrtiz]

Applied..thus closing this one.

[humitos]

For future reference, here is the commit:  http://git.sugarlabs.org/terminal/mainline/commit/f88b809dfb7c8f237ab7735f50ec4b8546ac4471