Skip to content

Conversation

pzrq
Copy link
Contributor

@pzrq pzrq commented Apr 10, 2024

Summary: update cupcake SVGs to handle light and dark modes

Description for the changelog:

update cupcake SVGs to handle light and dark modes

Other info:

Specifically:

  1. For cupcake alone, just add the white background-color
  2. For cupcake with threat dragon text, add three rectangles: a. Firstly, a black background for the white threat dragon text b. Secondly, a white background for the black cupcake c. Thirdly, a black background, so the 'T' in threat dragon has some contrast behind it as it overlaps the second white rectangle
  3. Translate the threat dragon text left a bit, so it doesn't overflow the SVG on the right side

closes #936

<--
Thanks for submitting a pull request!
Please make sure you follow our code_of_conduct.md and our contributing guidelines contributing.md
-->

Specifically:
1. For cupcake alone, just add the white background-color
2. For cupcake with threat dragon text, add three rectangles:
   a. Firstly, a black background for the white threat dragon text
   b. Secondly, a white background for the black cupcake
   c. Thirdly, a black background, so the 'T' in threat dragon has some contrast behind it as it overlaps the second white rectangle
3. Translate the threat dragon text left a bit, so it doesn't overflow the SVG on the right side

See #936
@jgadsden jgadsden self-requested a review April 10, 2024 15:36
@jgadsden jgadsden added enhancement New feature or request version-2.3 labels Apr 10, 2024
Copy link
Collaborator

@jgadsden jgadsden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all good, thanks @pzrq , and I will go ahead and merge

@jgadsden jgadsden merged commit 440be3b into OWASP:main Apr 10, 2024
@pzrq
Copy link
Contributor Author

pzrq commented Apr 10, 2024

@jgadsden Thank you and you're welcome.

@pzrq pzrq deleted the pzrq/cupcake-background branch April 10, 2024 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request version-2.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OWASP Threat Dragon - Logo in GitHub dark mode?
2 participants