-
Notifications
You must be signed in to change notification settings - Fork 81
Add official GitHub star count button above the fold #197
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
base: master
Are you sure you want to change the base?
Conversation
layouts/index.html
Outdated
@@ -7,6 +7,7 @@ | |||
</figure> | |||
|
|||
<div class="intro col-lg-7 col-md-16"> | |||
<a class="github-button" href="https://github.com/owncast/owncast" data-color-scheme="no-preference: dark; light: dark; dark: dark;" data-size="large" data-show-count="true" aria-label="Star owncast/owncast on GitHub">Star</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This label says "star owncast on github" but pressing this button will not star Owncast on github.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is designed to work like that only because of security issues- buttons/github-buttons#10
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know why, but I'm just saying it's not accurate. It's not possible to star a repository this way, so we shouldn't say that the button does so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be better to write "View owncast/owncast on Github" as the aria label and "Github" as button label?
fix #4280 add official GitHub star count button in hero section, which is above the fold