mckinsey / vizro

Vizro is a toolkit for creating modular data visualization applications.
https://vizro.readthedocs.io/en/stable/
Apache License 2.0
2.47k stars 109 forks source link

Restyle the documentation landing page #378

Closed stichbury closed 1 month ago

stichbury commented 3 months ago

This is a good issue for anyone with markdown experience who enjoys the styling/design aspect rather than writing copy. There's no knowledge of Vizro required.

The current landing page for documentation is rather bland https://vizro.readthedocs.io/en/stable/ and the presentation could potentially be improved.

It would be great to have some styling to improve it. We can start with the copy we have (although we do need to add a section to link to the demo example in the repo). Some basic redesign is all that is needed to boost the look and feel.

sulphatet commented 1 month ago

Hi, I would like to work on this, can this be assigned to me?

stichbury commented 1 month ago

Thank you for volunteering! We are making a few changes throughout our docs but one thing you could do, if you were interested in helping with changing the look of the docs, is to update the font across the docs to use Inter. We only use Inter regular, medium and semi-bold.

If you wanted to look at the relevant CSS changes for Vizro and Vizro-AI docs to replace the existing font with that one, and if you felt confident about doing that, we are happy to accept a contribution of that change. Beyond that, for now, we are going to pause any changes subject to some guidance from our design team.

WDYT? Would you like to give it a try? Thanks @sulphatet!

sulphatet commented 1 month ago

Hi @stichbury Yes! I would love to work on that.

From what I understand, I should be updating vizro-core/docs/stylesheets/extra.css and vizro-ai/docs/stylesheets/extra.css, is that right?

Thanks!

stichbury commented 1 month ago

I think it may be more straightforward than that, since we're using MkDocs Material theme. You can use https://squidfunk.github.io/mkdocs-material/setup/changing-the-fonts/ for guidance.

stichbury commented 1 month ago

This is done and merged. Thanks @sulphatet ! 🌟 🏆