Updated background and padding#19623
Conversation
Signed-off-by: Andre Wanlin <[email protected]>
github-actions
Bot
added
the
area:documentation
|
Uffizzi Preview |
camilaibs
left a comment
camilaibs
left a comment
There was a problem hiding this comment.
Thanks for updating the image 🙌🏻
Just left a question, nothing blocking this pull request 🙂
There was a problem hiding this comment.
Would it be possible to add a white background via CSS to all images as an alternative when the dark theme is active?
There was a problem hiding this comment.
Yeah, @vinzscam suggested that in the related issue. I went this route as I felt that getting it fixed quickly was better then having it be really hard to read. Maybe we could log a more general Issue with a help wanted label to add this?
There was a problem hiding this comment.
Makes sense, I agree with you. Do you mind opening up the more general issue for us 🙏🏻 ?
There was a problem hiding this comment.
Sure, did a quick look and found this: #17712. Would that work or should I log something new for this?
|
Thanks @freben and @camilaibs 🚀 |
3 participants
Hey, I just made a Pull Request!
Updated the permissions digram to have a white background and roughly 10px padding all around
Closes #19558
✔️ Checklist
Signed-off-byline in the message. (more info)