Skip to content

Conversation

@ldez
Copy link
Contributor

@ldez ldez commented Aug 24, 2025

Basically, the same idea as #659, but:

  • Disabled by default, with a new option to enable it
  • Icons based on Heroicons v1 and inside data/icons.yaml
  • Uses the partial to add the icons
  • Handles RTL and LTR languages
  • Uses Tailwind classes

Demo: https://deploy-preview-780--hugo-hextra.netlify.app/blog/v0.10/#technical-improvements

Closes #659

@netlify
Copy link

netlify bot commented Aug 24, 2025

Deploy Preview for hugo-hextra ready!

Name Link
🔨 Latest commit fd8ea85
🔍 Latest deploy log https://app.netlify.com/projects/hugo-hextra/deploys/68aaf7fcffa1c000086743e1
😎 Deploy Preview https://deploy-preview-780--hugo-hextra.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ldez ldez changed the title feat(links): feat: add an option to flag outbound links feat(links): add an option to flag outbound links Aug 24, 2025
@ldez ldez force-pushed the feat/outbound-links branch 3 times, most recently from afa21b2 to ca92d48 Compare August 24, 2025 01:53
@ldez ldez force-pushed the feat/outbound-links branch from ca92d48 to 65ecb83 Compare August 24, 2025 02:38
@ldez ldez changed the title feat(links): add an option to flag outbound links feat(links): add an option to flag external link decoration Aug 24, 2025
@ldez ldez changed the title feat(links): add an option to flag external link decoration feat(links): add an option for external link decoration Aug 24, 2025
Copy link
Owner

@imfing imfing 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!

@imfing imfing merged commit a19de79 into imfing:main Aug 24, 2025
4 checks passed
@ldez ldez deleted the feat/outbound-links branch August 24, 2025 11:57
@imfing imfing mentioned this pull request Aug 25, 2025
18 tasks
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.

2 participants