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

Add ability to disable automatic linking for Github-Style shortcodes #1099

Conversation

AutomationD
Copy link
Contributor

Fixes #1098

@vercel
Copy link

vercel bot commented Oct 26, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @HEIGE-PCloud on Vercel.

@HEIGE-PCloud first needs to authorize it.

…(backwards-compatible, default is still true)
@AutomationD AutomationD force-pushed the feature/add-ability-to-disable-image-linking branch from 089be62 to 3333179 Compare October 26, 2023 12:49
@AutomationD AutomationD marked this pull request as ready for review October 26, 2023 12:49
@AutomationD
Copy link
Contributor Author

@HEIGE-PCloud can you please review? Thank you!

@@ -182,6 +182,9 @@ cacheRemoteImages = false
# 是否启用阅读时间估计
enableReadingTime = true
# Display a message at the beginning of an article to warn the reader that its content might be outdated.
# Whether to enable images to be linked to themselves by default
Copy link
Contributor

@CXwudi CXwudi Nov 3, 2023

Choose a reason for hiding this comment

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

Make sure to add a Chinese translation of the English description, you can use any translator like DeepL

@@ -182,6 +182,9 @@ cacheRemoteImages = false
# 是否启用阅读时间估计
enableReadingTime = true
# Display a message at the beginning of an article to warn the reader that its content might be outdated.
Copy link
Contributor

@CXwudi CXwudi Nov 3, 2023

Choose a reason for hiding this comment

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

This comment at line 184 should stay together with the comment at line 188 as it is the English translation of that Chinese description

@HEIGE-PCloud
Copy link
Owner

I believe the link element there is mainly for the light gallery support. The right way to do it might be to disable the links when light gallery is not enabled.

@HEIGE-PCloud
Copy link
Owner

Fixed in 9e7d099

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.

[FEATURE] Ability to disable image linking by default
3 participants