-
Notifications
You must be signed in to change notification settings - Fork 267
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
Create libGD_logo-1.svg #793
Conversation
Project logo was made libGD_logo-1.svg
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.
you just traced the existing png log from the website right ?
<?xml version="1.0" encoding="UTF-8"?> | ||
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd"> | ||
<!-- Copyright Rafał Jopek ( rafaljopek at hotmail com ) --> | ||
<!-- created: 2021-11-20 --> |
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.
these date tags aren't useful. just delete them.
@@ -0,0 +1,49 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
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.
why libGD_logo-1.svg
? seems like logo.svg
would suffice.
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.
The very name "logo.svg" says nothing, suggest libGD_logo.svg
</style> | ||
</defs> | ||
<g id="libGD_logo-1"> | ||
<metadata id="libGD_logo-1"/> |
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 metadata tag isn't used, so delete it
<!-- Copyright Rafał Jopek ( rafaljopek at hotmail com ) --> | ||
<!-- created: 2021-11-20 --> | ||
<!-- updated: 2021-11-22 --> | ||
<svg xmlns="http://www.w3.org/2000/svg" xml:space="preserve" width="120mm" height="78mm" version="1.1" style="shape-rendering:geometricPrecision; text-rendering:geometricPrecision; image-rendering:optimizeQuality; fill-rule:evenodd; clip-rule:evenodd" |
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 think a bunch of these style values aren't needed, so trim them
<defs> | ||
<style type="text/css"> | ||
<![CDATA[ | ||
.fil6 {fill:#00AA4D} |
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.
should give these fill rules more descriptive names
Can we re open this PR? A SVG logo, even if it can improved, would be nice to have :) |
Project logo was made libGD_logo-1.svg