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

Automatic parsing feature parses page titles and page properties #86

Open
cullen-dev opened this issue Jul 18, 2024 · 0 comments
Open

Comments

@cullen-dev
Copy link

Original Issue: Linker parsing property blocks #28

Details:

Sep 23, 2022

Having the "autocomplete on Enter" on, the linker parses property blocks and tries to link them.
This corrupts the syntax and throws a parsing exception for the block.
E.g., type:: page becomes [[type]]:: [[page]]

Sep 24, 2022 - @andreoliwa's response:

Hi, this should have been fixed by #29. The new release made today includes the fix: https://github.com/sawhney17/logseq-automatic-linker/releases/tag/1.3.0


Update 2024-07-18

According to the information above, this issue should have been resolved by #29 however, it does not appear that this issue has truly been resolved. It may have been previously, but it appears that this behavior is still present. I have only tested this issue with the alias:: and title:: properties but I am assuming this applies to any and all page properties, but I have not specifically tested any others. The impact on these two properties alone is enough for it to be a breaking issue for me.

Example:

When attempting to create a new page called "mobile version" with the following properties:

alias:: mobile versions, mobile version's, mobile versions', mobile-version, mobile-versions
title:: mobile version

image

Upon pressing Enter key, the auto-linker plugin parses the page title and the alias values into page references. #29

alias:: mobile [[versions]], [[mobile [[version]]]]'s, [[mobile]] [[versions']], mobile-version, mobile-versions
title:: mobile [[version]]

image

Any aliases affected by this do not work as intended.
For example, the alias, "mobile versions", that I attempted to add, became "mobile [[versions]]". As a result, when I try to create a page reference to "mobile versions" using "[[mobile versions]]", it attempts to creates a new page, because it does not recognize it as an alias of the "mobile version" page:

image

I'm not sure if a more recent update broke the fixes implemented in #29, or if this is the intended behavior, but it is really frustrating. Obviously I can just turn off the auto-parsing feature when creating new pages/adding page properties, but I don't always think about it, especially when I am in a hurry. I don't see any personal use cases where I would want to create automatic links in my properties. In the case that I wanted auto-links in my properties block, I could use the hotkey for auto-linking an individual block. Any help/suggestions??!

I really love this plugin and it has saved me a ton of time! However, the current functionality is driving me just a little crazy.

Thanks in advance for any help!

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

No branches or pull requests

1 participant