Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: adopt a new theme: shibuya #3754

Merged
merged 5 commits into from
Feb 12, 2024

Conversation

BoboTiG
Copy link
Contributor

@BoboTiG BoboTiG commented Jan 30, 2024

What I did

Upgraded the documentation website to another level 🍾

It brings a few things:

  • Vyper code actually highlighted as Vyper code (and not Python)
  • JSON code actually highlighted as JSON code (and not JavaScript)
  • Set the proper code highlighter for shell commands
  • A copy icon on all code snippets to ease our lives
  • Use brand new logos 💇🏻 (check both light and dark theme: you will see the logo changing automagically 🪄)
  • A proper favicon 🥂
  • Remove custom CSS/JS code (cleaned up the config file too)
  • Remove unused Markdown parser (less dependencies)
  • Replace occurrences of vyper.readthedocs.io with docs.vyperlang.org
  • Fix the Read The Docs integration

Closes #3746.

How I did it

  • Updated the theme
  • Lots of stuff to fix minor issues here and there
  • Fixed the Read The docs integration (we lost the HTML download option, but that's a good compromise I would say)

How to verify it

Go check https://vyper-tests.readthedocs.io 😃

Commit message

docs: Adopt a new theme: Shibuya!

Description for the changelog

Migrated to the Shibuya theme for the documentation website.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

Copy link

socket-security bot commented Jan 30, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
pypi/shibuya@2024.1.17 network Transitive: environment, eval, filesystem, shell, unsafe +186 218 MB lepture
pypi/sphinx-copybutton@0.5.2 environment, filesystem, network Transitive: eval, shell, unsafe +186 218 MB choldgraf, ebp-bot

🚮 Removed packages: pypi/recommonmark@0.7.1, pypi/recommonmark@0.7.1, pypi/sphinx-rtd-theme@2.0.0, pypi/sphinx-rtd-theme@2.0.0

View full report↗︎

docs/conf.py Outdated
# so a file named "default.css" will overwrite the builtin "default.css".
html_theme = "shibuya"
html_theme_options = {
"accent_color": "plum",
Copy link
Contributor Author

@BoboTiG BoboTiG Jan 30, 2024

Choose a reason for hiding this comment

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

It is possible to use another color theme, see https://shibuya.lepture.com/customisation/colors/#accent-colors for available options.

I picked "plum" because it is the closer color to the new logo (and it actually renders very well).

I can test other colors, if you would like to.

Copy link

@pentcle pentcle Jan 30, 2024

Choose a reason for hiding this comment

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

Hello!

I just tried the colour palette from https://github.com/vyperlang/vyper-brand next to the radix palette you linked and I think Purple is the closest match, tonally:

image

Copy link
Contributor Author

@BoboTiG BoboTiG Jan 30, 2024

Choose a reason for hiding this comment

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

The color was updated from "plum" to "purple", and changes are live at https://vyper-tests.readthedocs.io. Thank you @pentcle!

Copy link

Choose a reason for hiding this comment

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

looks great, thank you @BoboTiG

@codecov-commenter
Copy link

codecov-commenter commented Jan 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (768b3e9) 84.78% compared to head (9492246) 84.81%.
Report is 5 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3754      +/-   ##
==========================================
+ Coverage   84.78%   84.81%   +0.02%     
==========================================
  Files          92       92              
  Lines       13139    13122      -17     
  Branches     2929     2933       +4     
==========================================
- Hits        11140    11129      -11     
+ Misses       1535     1530       -5     
+ Partials      464      463       -1     

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

@fubuloubu
Copy link
Member

How to verify it

Go check https://vyper-tests.readthedocs.io 😃

Not sure if it's just a config thing, but this test link is missing all of the older version docs + stable tag

@BoboTiG
Copy link
Contributor Author

BoboTiG commented Jan 30, 2024

Not sure if it's just a config thing, but this test link is missing all of the older version docs + stable tag

It's a config thing, it runs only from the PR branch for the demo.

@BoboTiG
Copy link
Contributor Author

BoboTiG commented Jan 30, 2024

I just checked, this new theme will be available only starting from the "latest" version. It won't automatically be backported to old versions. And old versions won't be lost, I just did not enable them in the test project.

Edit: I just enabled "master" to see how it looks.

@BoboTiG BoboTiG force-pushed the feat-new-doc-theme branch from ab1395c to 54265a5 Compare January 30, 2024 23:18
@charles-cooper
Copy link
Member

@BoboTiG please don't force push to PRs that are in the middle of being reviewed! it makes it very hard to keep track of changes.

@charles-cooper
Copy link
Member

i think the only confusing thing about the new theme is that the "sub" navbar moves to the right hand side of the page.
Screenshot from 2024-01-31 13-07-41
Screenshot from 2024-01-31 13-08-20

@BoboTiG
Copy link
Contributor Author

BoboTiG commented Jan 31, 2024

I'll see to restore the sub navbar on the left 👍🏻

- move the right one to the left
- adapt the main container to use the full width
@BoboTiG
Copy link
Contributor Author

BoboTiG commented Feb 4, 2024

How about now @charles-cooper?

@charles-cooper
Copy link
Member

hmm @BoboTiG, if it costs us the ability to expand the submenus, let's move the navbar back to the right.

@charles-cooper
Copy link
Member

can we also change the logo (on the top left) to the purple one?

@BoboTiG
Copy link
Contributor Author

BoboTiG commented Feb 7, 2024

You were right: the colored logo on the top-left looks cooler 👍🏻

Copy link
Member

@charles-cooper charles-cooper left a comment

Choose a reason for hiding this comment

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

cool. thanks!

@charles-cooper charles-cooper merged commit a2eb60c into vyperlang:master Feb 12, 2024
84 checks passed
@charles-cooper charles-cooper changed the title docs: Adopt a new theme: Shibuya! docs: adopt a new theme: shibuya Feb 12, 2024
@charles-cooper
Copy link
Member

i'm realizing we need lower contrast. for the light mode, it would be nice to have some kind of grey/beige/off-white as the background

(paging @pentcle )

@BoboTiG BoboTiG deleted the feat-new-doc-theme branch February 13, 2024 07:16
@lepture
Copy link

lepture commented May 21, 2024

There are updates for shibuya theme. Your current documentation has a UI issue: you can scroll down bellow the footer part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docs: Adopt a new theme?
6 participants