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: Add full-text content output #7

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

Conversation

Maecenas
Copy link

Add full-text content output.

Description

Add as a default option to export full-text content.

Checklist

  • All tests are passing
  • My code follows the code style and structure of this project

You may preview the sample feed output at my blog. One thing to notice is that I haven't exported the canonical src for <img> tags. Local development with Reeder 4 would correctly display the result (<img src='/pic.jpg'></img>), while Reeder 3 wouldn't. The default heading anchor (#) from VuePress also didn't been removed from the output.

Development (Reeder 4)
Development on Reeder 4
Development (Reeder 3)
Development on Reeder 3

@webmasterish
Copy link
Owner

Hi @Maecenas,

It needs to support any feed reader, that's why the paths need to be absolute.

@Maecenas
Copy link
Author

Maecenas commented Jun 2, 2019

Hi @webmasterish,
The images on third-party CDN work there. The images with absolute src can be addressed by calling url.resolve(path: string): string. But for those images with relative src (./test.jpg, those placed in the same folder with markdown), I have no idea for now, as plugins like this are applied before the minification started.

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