-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
readme: update readme logo's resolution #4713
Conversation
Signed-off-by: Calvin Weng <wenghao@pingcap.com>
Signed-off-by: Calvin Weng <wenghao@pingcap.com>
Signed-off-by: Calvin Weng <wenghao@pingcap.com>
Signed-off-by: Calvin Weng <wenghao@pingcap.com>
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.
LGTM
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.
Could you keep the new logo? I like it!
I also like our big obnoxious logo! |
@dcalvin It seems the new logo has a white background. Some folks use CSS skins to change how Github looks, do you think we could use a transparent background? |
* update latest logo Signed-off-by: Calvin Weng <wenghao@pingcap.com> * reverse file permission to 644 Signed-off-by: Calvin Weng <wenghao@pingcap.com> * fix logo-too-big issue Signed-off-by: Calvin Weng <wenghao@pingcap.com> * update logo's resolution in readme Signed-off-by: Calvin Weng <wenghao@pingcap.com>
* update latest logo Signed-off-by: Calvin Weng <wenghao@pingcap.com> * reverse file permission to 644 Signed-off-by: Calvin Weng <wenghao@pingcap.com> * fix logo-too-big issue Signed-off-by: Calvin Weng <wenghao@pingcap.com> * update logo's resolution in readme Signed-off-by: Calvin Weng <wenghao@pingcap.com>
What have you changed? (mandatory)
Please explain IN DETAIL what the changes are in this PR and why they are needed:
What are the type of the changes? (mandatory)
The currently defined types are listed below, please pick one of the types for this PR by removing the others: