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

feat(frontmatter): Add an option to index frontmatter wikilinks #1673

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

Conversation

necauqua
Copy link
Contributor

@necauqua necauqua commented Dec 24, 2024

I was missing this feature and implemented it, not knowing that there was #944 already ¯\_(ツ)_/¯

Looking at it now I think that my impl is a little better, and I'd love to have it upstreamed.
Ready to get review comments, I know the option name is meh.

Closes #820 and #944

Before

image

After (with indexFrontmatterWikilinks: true)

image

With my frontmatter component :)

image

@necauqua
Copy link
Contributor Author

(yes I forcepushed for minor typos 6 times 😂)

@necauqua
Copy link
Contributor Author

Here's an example of the PageProperties component for rendering frontmatter I'll probably make a PR for soon, for which I needed this among other things

image

I was missing this feature and implemented it, not knowing that there
was jackyzha0#944 already ¯\_(ツ)_/¯

Looking at it now I think that my impl is a little better, and I'd love
to have it upstreamed.

Closes jackyzha0#820 and jackyzha0#944
@necauqua necauqua force-pushed the index-frontmatter-wikilinks branch from cf82870 to de41860 Compare January 2, 2025 21:17
@Mikodin
Copy link

Mikodin commented Jan 9, 2025

Just want to add that this is huge for me and I'm grateful that you did this @necauqua 🙏

I added this to my project and have had no issues and it's working exactly like it does in obsidian

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.

Detect links in properties
2 participants