-
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Alt-N, goto-next-clone, does not select proper body text #61
Comments
@edreamleo Hmm, I cannot reproduce this bug. I did it with LeoPyRef exactly like you suggested... weird. Perhaps you could grab screenshot, or setup a zoom session if you can reproduce the bug reliably? |
On Mon, Dec 11, 2023 at 7:18 PM Felix ***@***.***> wrote:
Perhaps you could grab screenshot, or setup a zoom session if you can
reproduce the bug reliably?
I definitely can reproduce this both with desktop vs-code and vscode.dev on
Microsoft Edge.
In both cases, I have enabled LeoJS and *disabled *LeoInteg. To reproduce:
- Create a clone of a node containing text. It should not be a top-level
node.
- Move the clone so it is the last top-level node.
- Alt-N.
The problem doesn't *always* happen, but it usually does. Screenshot
below. The outline pane highlights the node BaseText.make_data, but the
body pane contains the text of the top-level "Code" node.
BTW, the tab name 'LEOJS: BODY' is useless. It would be much better to have
it display c.p.h.
I would be happy to zoom about this.
Edward
[image: image.png]
|
I definitely can reproduce this both with desktop vs-code and vscode.dev
on Microsoft Edge.
In both cases, I have enabled LeoJS and *disabled *LeoInteg.
I just re-enabled LeoInteg. I get the same failure with desktop vs-code.
Edward
|
@edreamleo Thanks for showing me a reproduction of the bud. I've just noticed also that Leo actually allows you to do this command on a non-clone node, in which case it just does a 'find next clone'. I'll start by fixing this behavior to match Leo and go on from this to make more experiments to try to squash this bug. |
On Sat, Dec 16, 2023 at 1:25 PM Felix ***@***.***> wrote:
@edreamleo <https://github.com/edreamleo> Thanks for showing me a
reproduction of the bud. I've just noticed also that Leo actually allows
you to do this command on a non-clone node, in which case it just does a
'find next clone'.
You're welcome.
I'll start by fixing this behavior to match Leo and go on from this to
make more experiments to try to squash this bug.
Happy hunting :-)
Edward
|
@edreamleo the devel branch now has a possible fix. Can you try it out on your machine ? (I'll send you a pre-packaged extension file if you prefer) |
@edreamleo I just found a way to reproduce it reliably on my machine: the other (target) clone to be navigated-to has to be hidden in a collapsed part of the tree. The goto-next-marked command has the same problem. I'm near finding the exact source of the problem for those command refresh-ui resolution bugs. |
@boltex I've pulled devel. How do I test the code? |
Just by launching the debugger with F5. But no need to do so now because I've found a way to reproduce! So dont bother since I can now test reliably by myself. (I proposed that you tried it out because i could not reproduce the bug earlier.. hehe.) |
@boltex Alright. I'll leave you to it. Good luck! |
Fixed in #70 |
@boltex The fix works for me on both vs-code and vscode.dev. |
This is a serious bug that might lead to corrupted data.
Using LeoJS in vscode.dev:
class NestedSplitterTopLevel (QWidget)
It appears that the body text is the text of the node's parent.
The text was updated successfully, but these errors were encountered: