adds a banner to README #205
Conversation
|
|
| @@ -1,4 +1,4 @@ | |||
| # Temporal Python SDK | |||
|  | |||
There was a problem hiding this comment.
Can you shrink the header height down considerably? I think it should be a quarter or less of what it currently is. There's no value in the precious vertical space wasted IMO. Yes that makes the snake smaller, but I think it's worth it if it's really a "banner".
There was a problem hiding this comment.
Cool, Figma had a "banner" output selection for a profile banner so that should work and make it smaller. Let me know if I need to size it down anymore.
There was a problem hiding this comment.
I didn't mean shrink the title text so small, I just meant the image size (sorry for being unclear). Can leave the title text the same size it was. And you may want to move the snake to the right a bit since the title may get a bit close to it.
There was a problem hiding this comment.
Looks great! Will make sure CI completes and merge shortly.
3 participants
What was changed
Adds a Python banner to Readme.
Why?
Repalces the #h1 title
Checklist
Closes
How was this tested: