[Geany-devel] Split Window Patches
enrico.troeger at xxxxx
Sun Mar 27 16:11:13 UTC 2011
On Sun, 27 Mar 2011 16:41:40 +0200, Enrico wrote:
>On Sat, 26 Mar 2011 22:26:42 -0700, Matthew wrote:
>>On 03/26/11 17:31, Colomban Wendling wrote:
>>>> Enable Scintilla's default popup editor menu on the split window
>>>> editor so that there's a way to copy/paste/undo/redo/etc.
>>>> Closes bug #2983145 although the keybindings and main menu items
>>>> still don't work in the split window.
>>> Not sure about this one since the menu is neither translatable nor
>>> look like other Geany's menus. Maybe manually implementing a basic
>>> menu would be better?
>>Should a bug/feature request also be reported to Scintilla?
>>>> Remove the widget reparenting in an attempt to make the plugin work
>>>> on Windows again. Although I can't test on Windows, I think
>>>> there's a good chance this will fix the issues and it seems to work
>>>> fine on Linux as well. I base this on an old GTK+ FAQ entry.
>>>> If someone can confirm, it will close bug #2725342 and the
>>>> plugin can be re-enabled on the Windows build.
>>> Not sure about this, what was the issues before (the one the code
>>> talked about)?
>>> I'd prefer one of the original authors to review this one if
>>> possible... Nick (or Enrico, seems you also worked on this), could
>>> you check this/tell what was exactly the issue?
>>See r3161 and r3163. also the bug report for this and FAQ entry
>>linked from original message. I think it's best to have original
>>authors (especially someone who is able to build on windows) to check
>I guess that means me :).
>I tested the patch (#0003) on Windows and it works fine, at least I
>couldn't find any wrong behaviour.
>The problems referenced in the code were about broken scrollbars, I
>don't remember exactly whether in the normal editor widget or in the
>splitted. And there were issues with the PRIMARY selection which
>behaved wrong. None of these I could reproduce with the patch #0003,
>neither on Windows nor Linux.
They still exist, at least the broken PRIMARY selection problem.
It's easy to reproduce: select some text in the normal editor widget
after you splitted it. You cannot paste the selected anymore by a
Colomban already reverted the two related commits. Thanks.
>I think we should commit it and see how it works for other people.
Er, let's say I changed my mind...:D.
Get my GPG key from http://www.uvena.de/pub.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: not available
More information about the Devel