Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@welpo
Copy link
Owner

@welpo welpo commented Sep 19, 2024

Summary

Enhances the "Visit website" link in the Atom feed to be section-aware and properly link to the language-specific main page when not in a section or tag.

Related issue

#393

Changes

  • Link to language-specific main site (used to always link to /)
  • If the feed belongs to a tag, it displays it on the link (see screenshot)
  • Add section/tag information to the Atom feed template
  • Update XSL transformation to conditionally display section/tag

Screenshots

feed link

Type of change

  • Bug fix (fixes an issue without altering functionality)
  • New feature (adds non-breaking functionality)
  • Breaking change (alters existing functionality)
  • UI/UX improvement (enhances user interface without altering functionality)
  • Refactor (improves code quality without altering functionality)
  • Documentation update
  • Other (please describe below)

Checklist

  • I have verified the accessibility of my changes
  • I have tested all possible scenarios for this change
  • I have updated theme.toml with a sane default for the feature
  • I have made corresponding changes to the documentation:
    • Updated config.toml comments
    • Updated theme.toml comments
    • Updated "Mastering tabi" post in English
    • (Optional) Updated "Mastering tabi" post in Spanish
    • (Optional) Updated "Mastering tabi" post in Catalan

@welpo welpo added the enhancement New feature or request label Sep 19, 2024
@netlify
Copy link

netlify bot commented Sep 19, 2024

Deploy Preview for tabi-demo ready!

Name Link
🔨 Latest commit ce5a23b
🔍 Latest deploy log https://app.netlify.com/sites/tabi-demo/deploys/66ecb931d3476f0008493b53
😎 Deploy Preview https://deploy-preview-394--tabi-demo.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 site configuration.

- Link to language-specific main page
- Add section/tag information to the Atom feed template
- Update XSL transformation to conditionally display section/tag
@welpo welpo force-pushed the feat/feed-visit-website-context-aware branch from d5bdfea to ce5a23b Compare September 19, 2024 23:52
@welpo welpo merged commit adfe28e into main Sep 19, 2024
8 checks passed
@welpo welpo deleted the feat/feed-visit-website-context-aware branch September 19, 2024 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant