crate / crate-docs-theme

A Sphinx theme for the CrateDB documentation.
https://crate-docs-theme.readthedocs.io/
Apache License 2.0
20 stars 5 forks source link

New <h1> headline font: Montserrat #465

Closed amotl closed 9 months ago

amotl commented 9 months ago

Introduction

With that patch, the theme started using the Montserrat font for <h1> headlines.

Preview

It can be inspected on this downstream build of the CrateDB Guide.

Thoughts

/cc @karynzv, @msbt, @seut, @matriv, @proddata, @hammerhead

msbt commented 9 months ago

@amotl we can use any font-weight available: https://gwfh.mranftl.com/fonts/montserrat?subsets=latin - current one is 700, happy to lower it to 600.

msbt commented 9 months ago

Reduced font weight, as suggested.

amotl commented 9 months ago

Thanks!

amotl commented 9 months ago
  • We already reduced total width, by decreasing letter spacing, with 4fd785c83850.
  • Reduced font weight as suggested, see GH-466.

Sorry to be a pain. I believe, now that we decreased total width by reducing the font weight, it might be appropriate to reset the letter spacing back to normal. We are sharing a few samples below. Maybe I'm wrong. wdyt?

image -- https://crate-docs-theme.readthedocs.io/

image -- https://cratedb.com/docs/guide/admin/sharding-partitioning.html

image -- https://cratedb.com/docs/guide/

msbt commented 9 months ago

@amotl currently it's -1.4px, I wouldn't go back to the default since the font really is wide. What about some sort of middle ground, like -0.5px?

amotl commented 9 months ago

Sounds good. I think we will need to see it in action if it is more pleasant, and does not add back too much total width.

msbt commented 9 months ago

@amotl can you use the inspect element tool from your browser for that?

Screeencast-letter-spacing.webm

ckurze commented 9 months ago

Done - Thank you :)