Conversation
|
Thanks @shellmayr , but, unfortunately I think we'll need to tweak this logo before merging. The new SVG uses a |
|
Ah yeah, I can fix that though and add a white background rect, is that a good approach? Alternatively I can replace the circle with a rect as well |
|
@priscilawebdev I added a white bounding rectangle - let me know if this works as an adaptation! |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Circle background leaves corners white, breaking icon consistency
- Replaced the circle background with a full-coverage rectangular fill using the brand color (#009688) in both svg/fastapi.svg and svg_80x80/fastapi.svg to match the consistent pattern used by all other icons.
Or push these changes by commenting:
@cursor push 3faa77fcc8
Preview (3faa77fcc8)
diff --git a/svg/fastapi.svg b/svg/fastapi.svg
--- a/svg/fastapi.svg
+++ b/svg/fastapi.svg
@@ -1,5 +1,4 @@
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20">
- <rect width="100%" height="100%" fill="#fff"/>
- <circle cx="10" cy="10" r="8.02" fill="#009688"/>
+ <rect width="100%" height="100%" fill="#009688"/>
<path fill="#fff" d="m9.45 5.62-1.6 3.56-.73 1.63-1.61 3.56 7.39-5.19H9.42l5.07-3.56Z"/>
</svg>
\ No newline at end of file
diff --git a/svg_80x80/fastapi.svg b/svg_80x80/fastapi.svg
--- a/svg_80x80/fastapi.svg
+++ b/svg_80x80/fastapi.svg
@@ -1,5 +1,4 @@
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 80 80">
- <rect width="100%" height="100%" fill="#fff"/>
- <circle cx="40" cy="40" r="32.09" fill="#009688"/>
+ <rect width="100%" height="100%" fill="#009688"/>
<path fill="#fff" d="m37.79 22.5-6.42 14.24-2.93 6.52-6.42 14.24 29.54-20.76H37.7l20.28-14.24Z"/>
</svg>
\ No newline at end of fileThis Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.
Reviewed by Cursor Bugbot for commit adf8b9e. Configure here.
priscilawebdev
left a comment
There was a problem hiding this comment.
Yes! I've noticed that other platforms, like Python, also use a white background, so this works well. Thank you! 🙏


FastAPI has had a new logo for a while, update it to the new color and shape
Closes TET-2222