Add white background to logo

[purva-thakre]

Description

Related to the work introduced in #416.

This PR makes sure the toqito logo is visible in dark mode of docs.

Inverted color version of the logo does not look that good. Could be tackled later in another PR as we want to be consistent with the color theme of wagtail. Added these images to try a customized version of CSS where the logo image is switched depending on which theme is being used: light/dark.

https://www.reddit.com/r/tailwindcss/comments/kd9czo/comment/gfw0zxb/?utm_source=share&utm_medium=web2x&context=3

Or we don't need to tackle this. Use the image with a white background.

Todos

Notable points that this PR has either accomplished or will accomplish.

• TODO 1

Questions

• Question1

Status

• Ready to go

[purva-thakre]

Update this link to fix broken image in PyPI.

https://stackoverflow.com/a/46875147/10241324

https://pypi.org/project/toqito/

[purva-thakre]

marking this ready for review even though I have 1 item I need to resolve.

I can't use a permalink to fix that comment because the new images I want to use are in this PR.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c0e7e7e) 98.0% compared to head (6ef24a2) 98.1%.
Report is 13 commits behind head on master.

Additional details and impacted files

@@          Coverage Diff           @@
##           master    #466   +/-   ##
======================================
  Coverage    98.0%   98.1%           
======================================
  Files         158     160    +2     
  Lines        3074    3107   +33     
  Branches      739     749   +10     
======================================
+ Hits         3015    3048   +33     
  Misses         37      37           
  Partials       22      22           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Bchass]

Did the inverted colors not work?

[purva-thakre]

yeah, they don't really look as clean as the current logo. Which is why I decided to remove them.

You can take a look at this commit: c663960

I think we can just keep the current logo for both light/dark modes. I added a white background for this reason.

[purva-thakre]

@vprusso Can I go ahead and merge this? I think merging this PR would be helpful for the WIP changes in #416.

I think I figured out why the image on the PyPI page is broken. This PR handles it and hopefully we will see the logo after a later release.

We can worry about customizing the logo for the new theme later.

[vprusso]

@vprusso Can I go ahead and merge this? I think merging this PR would be helpful for the WIP changes in #416.

I think I figured out why the image on the PyPI page is broken. This PR handles it and hopefully we will see the logo after a later release.

We can worry about customizing the logo for the new theme later.

Yep, merge away!