Attempt at GeeksForGeeks logo
Thank you for your contribution! Before sending this Pull Request, please confirm the following:
- [x] You have read the contributing guidelines
- [x] The icon's size is under 1,024 Bytes
- [x] The layout of the SVG looks like this including newlines
<svg xmlns="http://www.w3.org/2000/svg"
aria-label="..." role="img"
viewBox="0 0 512 512"><path
d="m0 0H512V512H0"
fill="#fff"/> ... </svg>
If you have done the above, please send the Pull Request.
this was my first attempt at creating a svg. hope i did ok
This is a great start! Just a couple comments. First, fill="none" instead of fill="#ffffff00" is shorter and even slightly better-supported. Second, you have enough small deviations from the original logo to be noticable. Fortunately, it looks pretty easy to get a lot closer. Below is an overlay of your version in purple and the original in green.
Moving the left circle left and right circle right by .5px:
Moving top points of polygons down 2px:
Moving top of line down 3px:
I don't know that those were the optimal changes, but it looks a lot better. I haven't tried implementing this in an efficient way, just playing around in Inkscape. You might also consider combining your two polygons into one <path>. There might be more clever <path> options for the green parts as well, but it would certainly be harder.
Oh, and if you can remove the last line break so the </svg> is on the third line, that'd be great!