EditPaste and Ctrl-V

2015-05-14 Thread lewis
Text copied from another program (e.g. text editor, LibreOffice) will not paste using menu commands EditPaste. Using keyboard Ctrl-V works fine. *Leo Log WindowLeo 5.1-final, build 20150514055303, Thu May 14 05:53:03 CDT 2015Git repo info: branch = master, commit = 24c99f770ee9Python 3.4.3,

Re: EditPaste and Ctrl-V

2015-05-14 Thread 'Terry Brown' via leo-editor
On Thu, 14 May 2015 07:30:32 -0700 (PDT) Chris George technat...@gmail.com wrote: Totally reproducible on Linux. You can add Cut to the list. Ctrl-X works fine, Cut from the menu has no effect. Delete works as advertised. Just a guess - command names from @cmd don't match menu definitions?

Re: EditPaste and Ctrl-V

2015-05-14 Thread Chris George
Totally reproducible on Linux. You can add Cut to the list. Ctrl-X works fine, Cut from the menu has no effect. Delete works as advertised. Chris Leo 5.1-final, build 20150514085122, Thu May 14 08:51:22 CDT 2015 Git repo info: branch = master, commit = 53c52549d5c7 Python 2.7.6, PyQt

g.command minor command naming issue

2015-05-14 Thread 'Terry Brown' via leo-editor
A minor thing, but the old def cmd_jump_to_error() code in leoscreen.py plugin would generate the command leoscreen-jump-to-error. The new g.command(leoscreen-jump_to_error) decorator is generating the command leoscreen-jump_to_error. Trivial solution is to change the decorator to

Re: EditPaste and Ctrl-V

2015-05-14 Thread lewis
Both menu commands EditCopy and EditPaste are broken. Using EditCopy in Leo has no effect. It will not replace anything previously copied to the clipboard. However using Ctrl-C works. Regards Lewis -- You received this message because you are subscribed to the Google Groups leo-editor group.

Re: Better @g.command decorator

2015-05-14 Thread Edward K. Ream
On Thu, May 14, 2015 at 5:04 AM, reinhard.engel...@googlemail.com wrote: So it looks like we're stuck. Any ideas? Apart from using a sophisticated decorator you could just pass the class name as an optional parameter in @cmd(command_name, class_name=None) I'm no expert here, but this

Re: EditPaste and Ctrl-V

2015-05-14 Thread Edward K. Ream
On Thursday, May 14, 2015 at 11:32:11 AM UTC-5, Edward K. Ream wrote: ​The problem is that ​g.isTextWrapper(w) returns False for the LeoQtBody object that is passed to LeoFrame.copyText, etc...So the decorator is not directly at fault. This problem existed on February 3, so recent work is

Re: EditPaste and Ctrl-V

2015-05-14 Thread Edward K. Ream
On Thu, May 14, 2015 at 9:40 AM, 'Terry Brown' via leo-editor leo-editor@googlegroups.com wrote: ​ ​ Just a guess - command names from @cmd don't match menu definitions? ​The problem is that ​g.isTextWrapper(w) returns False for the LeoQtBody object that is passed to LeoFrame.copyText, etc.

Re: g.command minor command naming issue

2015-05-14 Thread Edward K. Ream
On Thursday, May 14, 2015 at 9:30:17 AM UTC-5, Terry Brown wrote: I'm wondering if g.command should translate '_' to '-' automatically? I would prefer that everything is explicit, as much as possible. So no automatic translation. I'll do a regex search for '_' in decorators soon :-) And

Re: Better @g.command decorator

2015-05-14 Thread Edward K. Ream
On Thu, May 14, 2015 at 4:44 AM, reinhard.engel...@googlemail.com wrote: ​ ​ So it looks like we're stuck. Any ideas? ​ ​ This works for both Python2 and Python3: ​Excellent! Not sure why it works, but I see that it does :-) I'm in the midst of a big code reorg at present. When that is done,

Re: EditPaste and Ctrl-V

2015-05-14 Thread Edward K. Ream
On Thu, May 14, 2015 at 9:40 AM, 'Terry Brown' via leo-editor leo-editor@googlegroups.com wrote: On Thu, 14 May 2015 07:30:32 -0700 (PDT) Chris George technat...@gmail.com wrote: Totally reproducible on Linux. You can add Cut to the list. Ctrl-X works fine, Cut from the menu has no

Rev b9dbdaa replaces leo/core/leoEditCommands.py with leo/commands/*.py

2015-05-14 Thread Edward K. Ream
All tests pass regardless of the g.new_files switch. I don't expect major problems. Please report any problem immediately. The next step will be to complete the changeover by removing leo/core/leoEditCommands and remove all code disabled when g.new_files is False. This is usually a pretty

Re: EditPaste and Ctrl-V

2015-05-14 Thread Edward K. Ream
On Thursday, May 14, 2015 at 7:20:02 AM UTC-5, lewis wrote: Text copied from another program (e.g. text editor, LibreOffice) will not paste using menu commands EditPaste. Fixed at d27c7ef. EKR -- You received this message because you are subscribed to the Google Groups leo-editor group.

Re: Better @g.command decorator

2015-05-14 Thread reinhard . engel . de
So it looks like we're stuck. Any ideas? This works for both Python2 and Python3: from functools import wraps isPython3 = True def cmd(command_name): def inner_cmd(func): @wraps(func) def _decorator(*args, **kwargs): type0 = args and str(type(args[0])) or

Re: Better @g.command decorator

2015-05-14 Thread reinhard . engel . de
So it looks like we're stuck. Any ideas? Apart from using a sophisticated decorator you could just pass the class name as an optional parameter in @cmd(command_name, class_name=None) I'm no expert here, but this would require about 150 changes in the Leo code base and could be done by a