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

fix: display http(s) reflinks too #327

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

martyone
Copy link

Consider a node with:

:ROAM_REFS: http://www.example.net

And another node referencing the page with a (plain) link:

Lorem ipsum http://www.example.net ...

With the point inside the first node, the roam buffer lists the other node under the "Reflinks" section as expected, but the roam UI does not give any clue about the relation between these two.

This commit ensures that the http(s) links that have nodes with associated refs are converted to id based links of type ref, so that the UI can handle them.

Consider a node with:

	:ROAM_REFS: http://www.example.net

And another node referencing the page with a (plain) link:

	Lorem ipsum http://www.example.net ...

With the point inside the first node, the roam buffer lists the other
node under the "Reflinks" section as expected, but the roam UI does not
give any clue about the relation between these two.

This commit ensures that the http(s) links that have nodes with
associated refs are converted to id based links of type ref, so that the
UI can handle them.
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

Successfully merging this pull request may close these issues.

1 participant