I feel indifferent about adding a color to the rat, but we are going to add a color there's a couple of considerations:

1. It should work in both a light and dark theme on the website.
2. It should have good contrast to the background in both cases

I didn't like going too grey on the rat. Take a look at the colors on the actual rat logo on the website for some ideas. It took a while to find the right grey there, but we stuck with white on the demo.

I don't think adding colors to that demo is necessary.

Another question: Why does this logo isn't in ratatui::widgets::RatatuiLogo?

This one is a demo/example rather than a widget.

We should probably rename that already existing widget to RatatuiTextLogo or RatatuiText and add the rat logo as RatatuiLogo.

I think that the logo is the Ratatui word, while the Rat holding the terminal is the mascot. That's kinda how I think of it at least (and could be wrong here).

Yeah, that also makes sense if we think that way. Then how about adding this mascot as RatatuiMascot widget (or even just Rat :D)

Though I'm not sure if that's super useful. It could be nice just as a gag.

Codecov Report

Attention: Patch coverage is 99.09091% with 1 line in your changes missing coverage. Please review.

Project coverage is 92.9%. Comparing base (03066d8) to head (17f90b5).
Report is 7 commits behind head on main.

@@           Coverage Diff           @@
##            main   #1584     +/-   ##
=======================================
- Coverage   93.0%   92.9%   -0.2%     
=======================================
  Files         72      76      +4     
  Lines      16830   17176    +346     
=======================================
+ Hits       15660   15960    +300     
- Misses      1170    1216     +46     

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

Thanks for the great review

Here is the updated code with the updated result

Hello

I've updated the code with corrections :)

LGTM - I pushed a last change to make the terminal border color 237 because it's (close enough to) the dark mode color for macOS borders.

Thanks for sticking with this to get it over the line. This is one of those changes that definitely was always going to have some back and forth due to the subjective nature, and I appreciate you for the work you spent getting it over the line.