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

Remove ecosystem stuff and port to scientific-python-hugo-theme #413

Merged
merged 14 commits into from
Oct 21, 2021

Conversation

jarrodmillman
Copy link
Member

@jarrodmillman jarrodmillman commented Oct 12, 2021

Closes #357, closes #99, closes #117, closes #382. This is a big change, but it is fairly straightforward. It does two main things.

  1. Replaces the scipy sphinx theme with the numpy.org hugo theme. Since we plan to use the numpy.org theme for multiple sites, we created https://github.com/scientific-python/scientific-python-hugo-theme. I will create a PR to numpy/numpy.org to switch them to scientific-python-hugo-theme. @stefanv and I will maintain scientific-python-hugo-theme and will implement whatever numpy.org and scipy.org want. (numpy.org has a web team that we need to talk to about the theme stuff, but we plan to follow their lead and intend to provide any help they need/want.)

  2. Removes the ecosystem stuff. Essentially, I replaced https://scipy.org with https://scipy.org/scipylib and removed scipy.org/scipylib. Since I moved to a new theme this involved moving the files and changing them from rst to md files. I used pandoc and manually reviewed the results. This change broke the tables. To make things easier to edit, we implemented a table syntax that you can see in, for example, content/en/download.md.

@stefanv and I are more than happy to help maintain this site for the next few years as part of our work on the https://scientific-python.org project. I suspect it would be worth updating the text on the site. But I think it makes more sense to merge this and then to make additional changes. I would be happy to help coordinate this, implement things that the group wants, and review other PRs.

@ilayn
Copy link
Member

ilayn commented Oct 12, 2021

We can go over the details but thanks for this folks!

@tupui
Copy link
Member

tupui commented Oct 12, 2021

Wooo thanks, this looks great! Not sure how we should go about this. Maybe we should already strip it down to the bare minimum (removing FAQ, gallery etc. which are currently for NumPy or SciPy ecosystem). Then we can decide in other PRs about what to add.

If we want to keep this already, then I can take some time to help go over all files.

@jarrodmillman
Copy link
Member Author

I force-pushed the newsite branch from 13aa0c3 to 444a401. The only change I made was to use the official logo. Originally, I found the logo via a web search on some random site. @rgommers provided a link to the official logo (see https://github.com/scientific-python/scipy.org-new/issues/12).

Here is what I had before
oldlogo

Now it looks like:
newlogo

Here are the favicons with a dark and light theme:
darkbar
lightbar

The official logo has a transparent snake now (before it was white). I don't have a strong opinion, but I think the transparent snake makes the favicon hard to see with the dark theme. That is fine with me, I just wanted to point it out.

@tupui
Copy link
Member

tupui commented Oct 12, 2021

Personally I prefer the white snake. It's more readable. The favicon would also look better. Maybe we can add some white in the official logo?

@stefanv
Copy link
Member

stefanv commented Oct 12, 2021

The official logo has a transparent snake now (before it was white). I don't have a strong opinion, but I think the transparent snake makes the favicon hard to see with the dark theme. That is fine with me, I just wanted to point it out.

I'd fill that snake with white, at least for the favicon. Maybe we should consider for the footer logo as well.

@tupui
Copy link
Member

tupui commented Oct 12, 2021

I'd fill that snake with white, at least for the favicon. Maybe we should consider for the footer logo as well.

Yes also the GitHub icon could be white IMO. But we could see all these later. This is already so much better than what we actually have 😅

@rgommers
Copy link
Member

I think it's just a problem with the official logo SVG file. It should always be white. We should redesign the logo at some point anyway, or at least have a graphic designer clean it up. For now let's just do what looks best.

@jarrodmillman
Copy link
Member Author

@stefanv did some minor clean up of the logo: scipy/archive#2

I will update the logo here after that has been reviewed.

Copy link
Member

@tupui tupui left a comment

Choose a reason for hiding this comment

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

I went through all the files, not much to adjust 👍

config.yaml Outdated Show resolved Hide resolved
Makefile Outdated
hugo $(BASEURLARG)
touch public/.nojekyll

public/.nojekyll: html
Copy link
Member

Choose a reason for hiding this comment

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

Is this a leftover, or you plan on adding something else?

Copy link
Member Author

@jarrodmillman jarrodmillman Oct 13, 2021

Choose a reason for hiding this comment

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

@rgommers, @ilayn, @tupui I updated the Makefile in 09f2da9. I am not certain that it is correct, so it would be good if you could carefully review it. Also, I have some questions:

Deployments

  • Do we still want to deploy to the Enthought servers? Or should we use netlify or gh-pages?

Redirects

  • Do we still want to do all the redirects from the old Moin site?
  • Since this moves everything up a level, do we want to add redirects from scipy.org/scipylib/* to scipy.org/* now?

Copy link
Member

Choose a reason for hiding this comment

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

I'd say yes to redirects from scipylib/* and let's get rid of the Moin redirects - those are a decade old by now and shouldn't get many hits anymore.

Do we still want to deploy to the Enthought servers? Or should we use netlify or gh-pages?

Let's do it properly and deploy to netlify/cloudflare/gh-pages. I think GitHub Pages is the most practical for deployment, and Netlify for preview.

Copy link
Member

Choose a reason for hiding this comment

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

Of those, I think only Netlify supports easy redirects.

Copy link
Member

Choose a reason for hiding this comment

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

Ah yes, forgot about that. We should probably go with the non-free version then, so we can give anyone who wants to help maintain it access.

config.yaml Outdated Show resolved Hide resolved
config.yaml Outdated Show resolved Hide resolved
content/en/bug-report.md Outdated Show resolved Hide resolved
content/en/mailing-lists.md Outdated Show resolved Hide resolved
content/en/install.md Outdated Show resolved Hide resolved
content/en/install.md Show resolved Hide resolved
static/gallery/team.html Outdated Show resolved Hide resolved
static/images/README.md Outdated Show resolved Hide resolved
jarrodmillman and others added 3 commits October 13, 2021 13:05
Co-authored-by: Pamphile ROY <roy.pamphile@gmail.com>
Co-authored-by: Pamphile ROY <roy.pamphile@gmail.com>
@jarrodmillman jarrodmillman mentioned this pull request Oct 13, 2021
@jarrodmillman jarrodmillman mentioned this pull request Oct 14, 2021
@tupui
Copy link
Member

tupui commented Oct 15, 2021

If this is going in soon, as pointed in scipy/scipy#14267, maybe we should redirect to the dev doc for now.

@rgommers
Copy link
Member

I'd be inclined to merge it soon, and then we can iterate on it more quickly. Can you send a message to the mailing list about it @jarrodmillman? I don't expect much there beside a bunch of thank you's, but it's a change we should run by the list.

@jarrodmillman jarrodmillman force-pushed the newsite branch 2 times, most recently from a451733 to de29232 Compare October 15, 2021 23:30
@netlify
Copy link

netlify bot commented Oct 15, 2021

✔️ Deploy Preview for loving-hopper-c72a13 ready!

🔨 Explore the source changes: 56201aa

🔍 Inspect the deploy log: https://app.netlify.com/sites/loving-hopper-c72a13/deploys/617105781806ce0007a5d3a1

😎 Browse the preview: https://deploy-preview-413--loving-hopper-c72a13.netlify.app/donations

@jarrodmillman
Copy link
Member Author

There are two minor things I want to take care of before this is ready to merge. I will update this PR by the end of the day.

@rgommers
Copy link
Member

We should keep a link to the SciPy conference website - perhaps in the footer?

@jarrodmillman jarrodmillman mentioned this pull request Oct 20, 2021
14 tasks
@jarrodmillman
Copy link
Member Author

@rgommers @tupui @ilayn I believe this is "ready" to merge. I created issue #418 to track what we need to do to deploy the new site after this is merged. @stefanv and I will take care of the DNS stuff. We will set up draft.scipy.org once you merge this PR. And we will wait to update scipy.org until a decision is made on #418.

Copy link
Member

@ilayn ilayn left a comment

Choose a reason for hiding this comment

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

I'll just give others to have a one final look and then we can click the button, once again thanks a lot for the effort folks!

Copy link
Member

@tupui tupui left a comment

Choose a reason for hiding this comment

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

LGTM as well, thanks a ton!

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

Went through all the pages, content seems fine, all links seem to work, and it looks great. In it goes!

@rgommers rgommers merged commit 0f45433 into scipy:master Oct 21, 2021
astrojuanlu added a commit to astrojuanlu/packaging.python.org that referenced this pull request Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants