copy/paste does not work in if one has not revealed the edit toolbar once

[bert]

In the Terminal, ctrl-shift-c/v do not work as usual.

Also, after trying to paste by dragging a clipboard item out of the frame into Terminal, the frame broke (can't retract anymore).

[manuq]

Yes, all the accelerator changes on init are giving the following warning:

1306979684.059092 WARNING root: No gtk.AccelGroup in the top level window.

sent from sugar.graphics.toolbutton.ToolButton.

I should fix it.

[manuq]

Attached a fix, but I'm looking for a better one.

[erikos]

Actually looks like a Sugar issue:

- open terminal

- copy something with Ctrl+shift+c --->>> does not work

- open the edit toolbar

- copy something with Ctrl+shift+c --->>> does work

---

- open Browse

- copy something with Ctrl+c --->>> does not work

- open the edit toolbar (you can even close it again)

- copy something with Ctrl+shift+c --->>> does work

[manuq]

It happens only with the new toolbars, with the old ones it works as expected.

[erikos]

The issue with the new toolbars is that we rely on the toplevel window when we want to set the accelerator [1][2]. We do set it for the activity window. The toplevel window of the Copy-button is only the activity window after the toolbar has been revealed once and the accelerator is not set before.

[1] http://git.sugarlabs.org/sugar-toolkit/mainline/blobs/master/src/sugar/graphics/toolbutton.py#line39

[2] http://www.pygtk.org/docs/pygtk/class-gtkwidget.html#method-gtkwidget--get-toplevel

[godiard]

I have added a method in the Window class, to walk the objects in the toolbar_box and update the accelerators, and optional parameters activity in the methods to set the accelerators in the class ToolButton.

This patch resolve the problem in activities Terminal and Browse without changing the activities. If in one activity the programmer set the toolbar_box in the activity before adding all the buttons, must call the update_accelerators methods after add the controls.

Simon, tell me if you like this approach.

[godiard]

Simon,

I have looked at the code in sugar-toolkit/src/sugar/graphics/toolbarbox.py, and implemented a alternative solution using the 'hierarchy-changed' signal to pack the page when is possible. See patch 0001-Pack-page-in-ToolbarButton-when-is-conected-to-the-w.patch

This solution is more in line with the way used in toolbutton.py, but I don't like it because the signal is trigger many times and I think is suboptimal. The first patch can be less elegant but is not executing unneeded code all time. In fact with the first patch, we can remove the 'hierarchy-changed' signal handlers in toolbutton.py. See 0001-Remove-hierachy-changed-signal-handlers-in-toolbutto.patch

[erikos]

The only activity affected is Terminal it looks like. Browse and Write which have as well the new toolbar do work. This can be explained as Write is setting the accelerator to the non default values 'Ctrl+Shift+C'.

[erikos]

from Walter:

The problem is not only in Terminal. I have the same problem with
Turtle Art, where I don'[t try to override the shortcut. My workaround
had been to open and then close the edit toolbar on startup, but that
only seems to work intermittently these days.

See http://bugs.sugarlabs.org/ticket/2050
See http://bugs.sugarlabs.org/ticket/2751

[godiard]

Testing in TurtleArt and in Paint, I found is better use 0001-Pack-page-in-ToolbarButton-when-is-conected-to-the-w.patch

I can remove the workaround in TuirtleArt and continue working. My other patch does not work, and looking at the code in TurtleArt have sense, because the buttons and toolbars are created in different order. This patch does not depends of the programmer create the toolbars in a specific order.

[godiard]

Sent patch to sugar-devel

[erikos]

Pushed: http://git.sugarlabs.org/sugar-toolkit/mainline/commit/3a81ba74385881b8638e800cdfdbb73a3708e7af

Fix is in sugar-toolkit-0.92.3.1.g3a81ba7-1.fc14.olpc.i686.rpm

[erikos]

Is in 874.

Tested Write which works fine, TurtleArt does not start anymore, though.

[dsd]

We dropped this from 874 (erikos was testing a pre-release above) due the TurtleArt problem and the risk of there being more side effects. It is not a major bug; copy-paste is still available through more discoverable means, and does work with the keyboard in the majority of activities.

Ready for inclusion in 11.3.0 however! Thanks Gonzalo.

[dsd]

TurtleArt ticket filed as http://bugs.sugarlabs.org/ticket/2986

[erikos]

in sugar-toolkit-0.92.4-1.fc14.olpc.i686.rpm (in my public repo)

[dsd]

test in 11.3.0 build 1

[greenfeld]

Cutting & pasting using the keyboard shortcuts in Terminal and Write work without revealing the edit toolbar.

Cutting & pasting using the menu selections after using the keyboard shortcuts in Terminal and Write also work as well.

Also tested Browse as well.

[tonyforster]

OS882, ctrl-c ctrl-v do not work in Turtle Art till the toolbar has been revealed

[sridhar]

Related OLPC AU bug report: http://dev.laptop.org.au/issues/773