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

Implement filename extension in link completion #114

Merged
merged 1 commit into from
May 28, 2024
Merged

Implement filename extension in link completion #114

merged 1 commit into from
May 28, 2024

Conversation

RyanGreenup
Copy link
Contributor

These changes add two options to the config file:

  • include_md_extension_md_link
  • include_md_extension_wikilink

These default to false in order to preserve past behavior. When set to true completions will include the file extension, this is needed for compatibility with other software:

  • [[wikilink.md]]
    • required by Markor on Android
  • [some-link](some-link.md)
    • required by WikiJS, Marktext and I'm sure many others.

This seeks to close #113.

Please let me know if there are any specific changes you would like made.

Copy link
Owner

@Feel-ix-343 Feel-ix-343 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great for the most part!

README.md Outdated

# Whether wikilinks should include an extension or not (needed for Markor compatibility)
# for example [[File]] or [[File.md]]
include_md_extension_wikilink = true
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe we'd want to indicate this is false by default

README.md Outdated

# Whether markdown links should include an extension or not
# for example [File](file.md) or [File](file)
include_md_extension_md_link = true
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed, yeah. I'll do that now, one sec.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just changed that, I squashed the commit so it's the same message:

+ c2941e6...798653f include-extension -> include-extension (forced update)

@Feel-ix-343
Copy link
Owner

Also, do you have any feedback about the completion code? I am doing a refactor to simplify it right now.

Hardcode markdown extension

Create config entry and make markdown extionsion conditional

update README with new config options

update README so config reads as false by default
@Feel-ix-343
Copy link
Owner

🔥

@Feel-ix-343 Feel-ix-343 merged commit b39f500 into Feel-ix-343:main May 28, 2024
@RyanGreenup
Copy link
Contributor Author

Also, do you have any feedback about the completion code? I am doing a refactor to simplify it right now.

Nothing glaring, but I'll have a look through and let you know thoughts that come to me. I'm a data scientist by trade so your Rust is probably a lot better than mine, I'm keen on getting better with it though.

Where are you guys chatting, Element / Discord? Keep me in the loop, I love this project and would love to jump in where I can.

@Feel-ix-343
Copy link
Owner

Thank you for the kind words! Sounds good.

For now the discussion is on here. A few people use the discussions page to post feedback and ideas. Feel free to open issues too.

I don't think I'd start a discord yet, but it could be useful for the future. What do you think!

@RyanGreenup
Copy link
Contributor Author

Sounds great, I'll keep an eye on this project and stay around. You've done an excellent job here, good work.

@Feel-ix-343
Copy link
Owner

Ahh thank you!

I look forward to working with you in the future

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

Successfully merging this pull request may close these issues.

Include Extension on link insertion
2 participants