PR #4624: now in the devel and 6.8.9 branches

18 views
Skip to first unread message

Edward K. Ream

unread,
Apr 21, 2026, 3:31:30 AM (10 days ago) Apr 21
to leo-editor
PR #4624 makes all commands work when executed from Leo's menus.

And it does a lot more behind the scenes. The details are in the PR.

Please continue testing and report any problems.

And one more thing. A new method, LeoQtGui.create_wrapper_for_widget, contains defensive code that prints a warning to the console if all is not as expected. Please report such traces if you see them. Leo should work just fine in any case.

​Edward

P.S. I'm quite surprised by how much work it took to fix the bug to my satisfaction. The details are bewildering. Nevertheless, I am confident that the new code is significantly better than the code in Leo 6.8.8.

EKR

Edward K. Ream

unread,
Apr 21, 2026, 10:58:47 AM (10 days ago) Apr 21
to leo-editor
On Tuesday, April 21, 2026 at 2:31:30 AM UTC-5 Edward K. Ream wrote:

PR #4624 makes all commands work when executed from Leo's menus.

Big sigh. I've just created #4626: copy does not work in the Completion tab.

I'm not sure it ever worked. And  create_wrapper_for_widget does not issue any warning.  Btw, that method is mostly misnamed--it's supposed to return an existing wrapper by default, creating a wrapper only if something is amiss.

Edward
Reply all
Reply to author
Forward
0 new messages