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

Revamp entire website #19

Merged
merged 84 commits into from
Aug 25, 2020
Merged

Conversation

pradyunsg
Copy link
Collaborator

@pradyunsg pradyunsg commented Jul 29, 2020

Closes #17

This brings in the changes from https://github.com/pradyunsg/sphinx-themes/ into this project, replacing the entire existing project tree with that repository. Notable changes to that repository since last discussion:

  • Significantly cleaned up generation + CI pipelines
  • A completely rewritten README + CONTRIBUTING guide
    • I spent wayy too much time on this, but this covers basically everything so that's good. :)
  • conf.py files are included and linked-to from the gallery
  • CI builds now include the "generated-site" as an artifact that can be downloaded
  • Relicensed to BSD-2-Clause (like the current repository)

Post-merge tasks for maintainers/admins:

  • Change "branch for GitHub Pages", from "master" to "gh-pages".

Notably, this PR technically has fewer themes than master, but I say we add them in a follow up because... well, I'd like to do the "add-the-themes" task later and leave a bit of space for others to pick up some of the themes as well. :)

More consistent with noxfile.py
Also, call out to a script to do this task, to keep noxfile from getting
too complicated.
This is to accomodate for "default" themes
We'll add them manually one-by-one since sphinx-themes seems really bad.
- Breakup _build_theme and inline calls into  render_themes
- Allow for better error handling
- Blacken code
Also lets me get rid of that annoying options key that wasn't being used.
This shouldn't happen online, but is for development
@schettino72
Copy link

👍 IMHO A single-column layout (for large screens) would work better.

@pradyunsg
Copy link
Collaborator Author

pradyunsg commented Jul 29, 2020

I tried that locally yesterday, but it didn't look right to me on a 15" laptop screen or on my 34" monitor.

Having a single column means that all the width goes towards to making the images bigger, something that I didn't feel was worthwhile given that it's possible to get a good idea for how a theme looks even at half width and if you really need a closer look, there's a preview link available. :)

@schettino72
Copy link

Well, dont need to use 100% of the screen width.
i.e. It is just easier to compare the look on mobile when all images are aligned vertically.

Anyway, just my 2 cents. Thanks for great work.

@pradyunsg
Copy link
Collaborator Author

@shirou Nudge.

@shirou
Copy link
Contributor

shirou commented Aug 25, 2020

ah, sorry for late response. OK. let's merge this. but I want to replace the current website after adding themes as much as possible.

Thank you for your great contribution!!

@shirou shirou merged commit 5b992d2 into sphinx-themes:master Aug 25, 2020
@pradyunsg pradyunsg deleted the revamp-entire-website branch August 25, 2020 11:15
@pradyunsg
Copy link
Collaborator Author

Whee! Thanks @shirou! ^>^

I'll add the additional sites in a few follow up PRs! Thanks for the push access here. :)

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.

Thoughts on a revamp?
4 participants