-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
List the named logos in the frontend #1794
Conversation
Good call. I suspect #1762 may drag on for a bit, so this is good for for an interim solution. One small thing is that the long list does push the content out very wide in a fullscreen window: before: after: Do you think its worth putting this in a |
Generated by 🚫 dangerJS |
Good call. I've updated in d7065ee and it'll be deployed here in a moment: https://shields-staging-pr-1794.herokuapp.com/ |
Thanks! |
This probably could be accomplished using a webpack config, however this approach is consistent with the way we handle badge examples and I like that it's very obvious what's happening.
I like @chris48s's idea of putting these on their own page, though we should resolve #1762 first. This seems like a good incremental improvement.
Fix #1787