Skip to content
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

Note ID prevents proper nesting of lists. #173

Open
MahmoudFawzyKhalil opened this issue Dec 17, 2020 · 5 comments
Open

Note ID prevents proper nesting of lists. #173

MahmoudFawzyKhalil opened this issue Dec 17, 2020 · 5 comments

Comments

@MahmoudFawzyKhalil
Copy link

@MahmoudFawzyKhalil MahmoudFawzyKhalil commented Dec 17, 2020

image
image

Putting a line break fixes it

image
image

But running scan vault again resets it and removes the line break

@Pseudonium
Copy link
Owner

@Pseudonium Pseudonium commented Dec 17, 2020

Right, I might be able to do a special case for this. At the moment, the script automatically replaces ‘double new line then ID’ by ‘single newline then ID’, since it looks nicer usually.

@MahmoudFawzyKhalil
Copy link
Author

@MahmoudFawzyKhalil MahmoudFawzyKhalil commented Dec 17, 2020

Thank you for your prompt reply. I look forward to a solution or some sort of work around.

BTW this addon/plugin is incredible. Thank you for your hard work!

@Pseudonium
Copy link
Owner

@Pseudonium Pseudonium commented Dec 18, 2020

Hmm, I've tested this with some online markdown editors, and this doesn't seem to break nested lists. I wonder if this could be a bug in Obsidian's markdown to HTML engine? I'll file this as a bug report there for now.

@Pseudonium
Copy link
Owner

@Pseudonium Pseudonium commented Dec 18, 2020

Alright, here's the bug report on the Obsidian forum

@RoamanEmpire
Copy link

@RoamanEmpire RoamanEmpire commented Mar 31, 2021

I'm also experiencing this and it is pretty annoying since I like to use lists + folding to get some extra "active recall".

Currently I've modified the plugin code locally to add two new line lines before the ID, although it is a bit ugly.

@Pseudonium as a temporary fix, do you think it would be possible to (have the option to) move the ID to the top of a note instead of the bottom. Example:
image

Alternatively, since I'm still unfamiliar with the codebase and typescript, could you point me to the section of the code where this change can be implemented? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants