-
Notifications
You must be signed in to change notification settings - Fork 190
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
If an article doesn't have title, there's no link to the article #758
Comments
I am currently working on a frontend rewrite in vue and will fix this issue there. I dont think it is worth fixing this in the current Code |
@JonathanTreffler Do you have the issue ID for the reference? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Maybe relevant issue to pay attention to during rewrite: In some cases it happens that the
|
Duplicate of #2037? |
This has become an issue for Mastodon feeds, since they've removed their titles: mastodon/mastodon#18640 |
Yea that's very unfortunate, there has been no movement on the front-end part unfortunately. And as the frontend works even less with the recently released Nextcloud 27. |
Is this fixed in the new frontend @Grotax ? |
No idea, haven't seen such a feed yet. Without an example it's hard to tell what would happen. |
I will check when I have time. |
Not fully fixed, the Feeds from Mastodon for example do not provide a title at all. I guess some improvement could still be made here, to display something there and link it to the url of the item. In the DB the field should still be null since it reflects the feed as is. For other case idk, we need example feeds. |
Changed - add region role to content list and details so that it can be used by screen readers (#2946) - split display settings in "display mode" and "split mode" to allow combining modes (#2950) - add display mode optimized for screen readers (#2950) Fixed - use appropriate semantic HTML elements for the item list to be recognised by screen readers (#2946) - make title in item list clickable for screen readers to select the current item (#2946) - remove mime type check when importing `opml`, they are not reliable anyway (#2951) - Embedded images are all scaled to max (#2945) - If an article doesn't have title, there's no link to the article (#758) Signed-off-by: Benjamin Brahmer <[email protected]>
Changed - add region role to content list and details so that it can be used by screen readers (#2946) - split display settings in "display mode" and "split mode" to allow combining modes (#2950) - add display mode optimized for screen readers (#2950) Fixed - use appropriate semantic HTML elements for the item list to be recognised by screen readers (#2946) - make title in item list clickable for screen readers to select the current item (#2946) - remove mime type check when importing `opml`, they are not reliable anyway (#2951) - Embedded images are all scaled to max (#2945) - If an article doesn't have title, there's no link to the article (#758) Signed-off-by: Benjamin Brahmer <[email protected]>
Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)
Is your feature request related to a problem? Please describe.
If an article doesn't have title, there's no link to the article. Technically it's there, but as the text inside <a></a> tag is empty, it's not visible and not clickable.
Describe the solution you'd like
Update timestamp on the right side to make it a link to original article, or/and use a placeholder like "(no title)" when title is empty in order to make it visible and clickable.
Additional context
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: