-
Notifications
You must be signed in to change notification settings - Fork 21
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
Feature Request: Notes attached with code #16
Comments
I have the same problem and want to add this feature, but I have questions about how to follow the line. |
@zhaobenx Maybe can try to add a function to let the user edit the line number? Will it be easier than match the code?! |
Yes i believe so.
Thank you for your revert.
…On Sat, 7 Jan, 2023, 3:00 pm Hsuan, ***@***.***> wrote:
@zhaobenx <https://github.com/zhaobenx> Maybe can try to add a function
to let the user edit the line number? Will it be easier than match the
code?!
—
Reply to this email directly, view it on GitHub
<#16 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATGLCYCYT2RRT623FI6INELWREZRZANCNFSM46B2OYGA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
One helpful thing would be to automatically copy selected line(s) to the note, so even if line number would change, then any notes would be still relevant to part in note. Extension might try to automatically try to match it with appropriate part in the file, but just mere copying of content at the time would be helpful. |
Use a UUID, like See similar libraries: Both of these libraries implement this function but they have other issues that make them untenable. I'm a big fan of the way your extension renders in the gutter. Too bad it's not clickable. Of the three, your extension comes out on top. I also appreciate how you receive and respond to feedback. Pros/ConsMarginaliaExample: https://storage.googleapis.com/com-f5-parm.appspot.com/parm/images/VqfNjKOA6xom0VpRAbf8.quicktime
Does not:
SidenoteDoes:
Does not:
Some possible implementationsOption 1Attach UUID at the end of note line. Cmd + Click action UUID to navigate to note. Store Note: md footer could alternatively be defined as:
And it will still not render because these are not valid Let me know your thoughts. I haven't looked at the source but I've always been interested in working on a VS Code extension. |
@imHVD it ended up being a lot more work than I expected (I ended up having to rewrite half the code) but I got it working. I submitted a PR. We'll see if it gets reviewed. |
Hi, maybe this vscode plugin :https://marketplace.visualstudio.com/items?itemName=hurly.separable-notes can solve this problem. |
Feature Request: The note stays with the code and not the line.
After a note is added but the code is moved up or down few lines then the note should also move with that code. This way the note stays attached with the code, even when the codes is moved when new code is added above it changing the line originally at which the note was added
The text was updated successfully, but these errors were encountered: