-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix: remove background on octocat #452
Conversation
I don't agree on this change, it looks like octocat is just floating in midair. The background makes it look like its a "fold" opening. I'd be open to changing the color to something other than hard black though. |
I looked into first to see if I could make a line instead that replaced the 45 ° border. Could that be a thing?On 11 Aug 2023, at 10:35, Daniel Berg ***@***.***> wrote:
I don't agree on this change, it looks like octocat is just floating in midair. The background makes it look like its a "fold" opening. I'd be open to changing the color to something other than hard black though.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were assigned.Message ID: ***@***.***>
|
Line could work yeah |
Tried creating a path in Inkscape, but did not work. Just pushed it for reference. Who thought SVG paths could be so difficult. 🤣 |
55b1a0d
to
ffa6312
Compare
@roosta after some playing here https://developer.mozilla.org/en-US/play, I think I got it. 😄 |
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.
Perfect
It might be hard to see, but the black background is IMHO unnecessary. This pull request removed the path rendering the background.