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.

Sign up for GitHub

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

Jump to bottom

Feed items with no title #6218

Closed
thecptrgod opened this issue Mar 20, 2024 · 5 comments · Fixed by #6240
Closed

Feed items with no title #6218

thecptrgod opened this issue Mar 20, 2024 · 5 comments · Fixed by #6240
Labels
Feature Request ideas for new features
Milestone
1.25.0

Comments

@thecptrgod
Copy link

Describe the bug
Feed items that have no title parse with a random string of the URL instead of just displaying the first line of text. (As far as I know title is not required in the RSS spec.)

To Reproduce
Steps to reproduce the behavior:
Subscribe to a feed that doesn’t have titles for some or all feed items such as scripting.com’s feed. Then look at the feed in the reader.

Expected behavior
Ignore the empty title and just put body text in there.

Screenshots
Current behavior:
IMG_2921

Example of expected behavior (NetNewsWire on iPhone)
IMG_2920

Environment information (please complete the following information):
FreshRSS running via Docker on a RasPi 400. Accessed either through the web client on Chrome or through NetNewsWire syncing on my phone.
@Alkarex
Copy link
Member

Alkarex commented Mar 20, 2024

The fallback for now when there is no title is GUID or URL.
The body can also be blank (quite typical for e.g. image-oriented feeds)

@math-GH
Copy link
Contributor

math-GH commented Mar 20, 2024

Ref #4946

@math-GH math-GH added the Feature Request ideas for new features label Mar 20, 2024
@math-GH math-GH changed the title [BUG] Feed items with no title do not parse correctly Feed items with no title Mar 21, 2024
@math-GH
Copy link
Contributor

math-GH commented Mar 21, 2024

@thecptrgod How does it look like in FreshRSS? Your screenshots are from a mobile app.

@thecptrgod
Copy link
Author

@thecptrgod How does it look like in FreshRSS? Your screenshots are from a mobile app.

Screenshot_20240322_045541

FreshRSS web

Screenshot_20240322_045756

Feedly for comparison

@math-GH math-GH mentioned this issue Mar 27, 2024
Merged
2 tasks
@math-GH
Copy link
Contributor

math-GH commented Mar 27, 2024

Feedback to this PR is welcome: #6240

@Alkarex Alkarex added this to the 1.25.0 milestone Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request ideas for new features
Projects
None yet
Milestone
1.25.0
Development

Successfully merging a pull request may close this issue.

Empty title guid or first words math-GH/FreshRSS
3 participants
@Alkarex @math-GH @thecptrgod