Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds a "Repository of the Day" badge from Trendshift to the README.md file. The badge is positioned below the existing Discord badge and before the Table of Contents section.
Changes:
- Added Trendshift badge to showcase the repository's trending status
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| [](https://discord.gg/5Hus6fM) | ||
|
|
||
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> |
There was a problem hiding this comment.
The use of target="_blank" without rel="noopener noreferrer" creates a security vulnerability. When opening links in a new tab without these attributes, the new page gains access to the window.opener property, which can be exploited for phishing attacks or other malicious purposes. Add rel="noopener noreferrer" to the anchor tag to prevent this security issue.
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> | |
| <a href="https://trendshift.io/repositories/11301" target="_blank" rel="noopener noreferrer"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> |
|
|
||
| [](https://discord.gg/5Hus6fM) | ||
|
|
||
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> |
There was a problem hiding this comment.
The alt text contains URL-encoded characters (%2F) which makes it less readable for screen readers. The alt text should use plain text format for better accessibility. Consider changing "recloudstream%2Fcloudstream | Trendshift" to "recloudstream/cloudstream | Trendshift" or a more descriptive text like "Trendshift Badge for CloudStream Repository".
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> | |
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="Trendshift badge for recloudstream/cloudstream repository" style="width: 250px; height: 55px;" width="250" height="55"/></a> |
|
|
||
| [](https://discord.gg/5Hus6fM) | ||
|
|
||
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> |
There was a problem hiding this comment.
The inline style attribute should be removed. The width and height are already specified using HTML attributes, making the style attribute redundant. Additionally, inline styles can be problematic for users who apply custom stylesheets or use reader modes. The HTML width and height attributes are sufficient for specifying image dimensions.
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" style="width: 250px; height: 55px;" width="250" height="55"/></a> | |
| <a href="https://trendshift.io/repositories/11301" target="_blank"><img src="https://trendshift.io/api/badge/repositories/11301" alt="recloudstream%2Fcloudstream | Trendshift" width="250" height="55"/></a> |
No description provided.