Rewrite to a sphinx project and use unified theme by Cadair · Pull Request #698 · astropy/astropy.github.com · GitHub

You can see a preview of this here: https://astropy-org--698.org.readthedocs.build/

Fun things to checkout:

light and dark themes
the search!!

I think this is now ready for review, but please confine review to the diff in this PR. i.e. not the theme which should be reviewed here: astropy/astropy-sphinx-theme#48

HTML Pages left to port:

announcements/*.html
contribute.html
credits.html
team.html
paper1_10k_citations.html

Todo before merge:

Find someone with power to move the astropy.org domain to RTD punted to later
Be ready to reconfigure Github pages for actions upload
Add back Google Analytics, and start a discussion somewhere about improvements later
Merge theme PR and release astropy-sphinx-theme

Other potential improvements:

Use intersphinx for all links to core documentation
See if we can generate a dark theme variant of the home page background

I'll keep a list of things I think we could tidy up here as I go along:

We should get rid of all the affiliated package javascript and have sphinx just do all the page gen on build. Then cronjob the site to rebuild each day or week or something.
The Copy bibtex js on the citation page inlines all the bibtex which is just horrible, we should have sphinx read that in from somewhere sane. What is the canonical source of the citation bibtex?
The javascript for the team page should all be removed and replaced with build-time Python code.

You will have to pull this change in too after it is merged. FYI

Filter out astropy core package from pyOpenSci Affiliated listing #699

@astropy/astropy-website-maintainers This is now ready for review.

We should probably cut a release of astropy-sphinx-theme before merging this PR 😉

Can you pls cross link PR from that branch here for future reference? Thanks!

It's in the PR description above.

Oh right, nevermind. We cannot merge until this is resolved. I better block merge given the approval.

This is now ready for review

Shouldn't I wait for you to resolve #698 (comment) first?

This is now ready for review

Shouldn't I wait for you to resolve #698 (comment) first?

No, I don't think so. This PR is not about the theme it's about rewriting the astropy.org site to use sphinx. Comments about the theme should go on the theme PR.

Now for the record: I'm not a maintainer of this repro because I have deep experience with web development, it's because we used to manually update the list of affiliated packages in this repro and I was the affiliated package editor at the time (and are again now, though we know how've a difference mechanism).
@eteq is the person who usually reviews changes like this.

That said, I'll have a look as soon as I can.

Most of the changes are pretty straightforward changes for html->rst. I'm not saying that to diminish @Cadair 's work (I know it's still a lot of work to do that, but it's not conceptually hard), just to say that the review is basically "it works" and there is no need to review every changed line, because we would not want to include any changes in the wording of individual pages here anyway.
Otherwise, it's a pretty standard setup and I see no problems with it.

Not really important here, since this is just a comment anyway, but I think most modern browsers now refuse to load any local file for security - for testing locally you'll a local web server running for all or most of them.

yeah this comment is pretty old 😆

I'd like to purge this entire file of js if I have the time.

You have to resolve conflict to pick up #699 . Thanks!

Merged in main

Hmm something is wrong.

Looks like that's a circleci problem, the way that circleci previews somethings doesn't really work properly. If you look at the RTD preview it seems to work.

Thanks for the clarification. I hope you are right...

So this is good to merge or are we waiting on something else?

Can you please add comment to explain what this is for, for future reference?

Also... Don't we also use some sort of banner mechanism here that "talks" with astropy cord RTD, like when we announced the 10k citation stuff. Would that still work?

Oh right, nevermind. We cannot merge until this is resolved. I better block merge given the approval.

So this is good to merge or are we waiting on something else?

There's a todo list in the OP, but mostly it's blocked on people reviewing / approving / arguing about colours on the theme PR.

Waiting for sunpy theme release

Also... Don't we also use some sort of banner mechanism here that "talks" with astropy cord RTD, like when we announced the 10k citation stuff. Would that still work?

Yes, but I should actually make this site display the banner if it's active!

use intersphinx for references to docs.astropy.org

@eteq I think the latest commit f75fd07 (this PR) adds a workflow which can be configured to publish the built site to github pages.

I think there's still some configuration work to be done to make that actions based deployment work (github actions environment etc).

I'm generally happy with all of this, but I have a few questions about finalization steps, @Cadair :

The changes reminded me our release announcements are generally rendered here. I'm not sure how much we're using that in the more recent releases (@astrofrog might know?), but it's possible that some links will be broken by this if e.g. some of them assumed the github url instead of the astropy.org URL?
@Cadair, I'm not sure what the finalization was you had in mind here as your last few posts suggest different things to me: are you thinking to use the gh-actions build to mirror the RTD build to here and have this remain the "true" version? (which would then require no DNS change I think) Or just have this as an alternative build and still update the DNS to point to RTD?

In some out-of-band discussion, @Cadair clarified the answer is that the desired end-state is RTD only. But we want to take a phased approach since we're not sure what all might break in the transition. So the plan is:

Reconfigure astropy.github.com to build gh-pages from actions (in preperation for merge).
Merge the PR
See that astropy.org continues to work hosted by gh-pages
@eteq updates teh DNS
figure out what we need to do to disable gh-page or if we can't put a placeholder page up

I'll do 1/2 momentarily, and then we'll follow up with the rest elsewhere (since this will be merged by then)

I believe I have done step 1, so merging now

One additional step I realized was also needed: disableing the circleci webhook since it's not used post-merge.

Cadair force-pushed the unified-theme-rewrite branch from 4b2fc98 to af49c3f Compare January 19, 2026 16:49

Cadair force-pushed the unified-theme-rewrite branch 4 times, most recently from d351274 to d4640ba Compare January 20, 2026 17:17

This comment was marked as outdated.

Sign in to view

Cadair marked this pull request as ready for review January 20, 2026 17:20

Cadair changed the title WIP: Rewrite to a sphinx project and use unified theme Rewrite to a sphinx project and use unified theme Jan 20, 2026

Cadair mentioned this pull request Jan 20, 2026

Add substitutions to credits.rst for astropy.org site astropy/astropy#19192

Merged

1 task

Cadair commented Jan 21, 2026

View reviewed changes

Cadair mentioned this pull request Jan 21, 2026

Tracking Issue: Unified Astropy Sphinx Theme Migration astropy/astropy-project#522

Open

10 tasks

hamogu approved these changes Jan 21, 2026

View reviewed changes

pllim reviewed Jan 27, 2026

View reviewed changes

Comment thread .circleci/config.yml Outdated

pllim reviewed Jan 27, 2026

View reviewed changes

pllim requested changes Jan 27, 2026

View reviewed changes

Cadair added 13 commits February 5, 2026 16:07

TMP: help

d6e4fb6

TMP: history

dce9fc5

TMP: team

2f9da79

TMP: Contributing

8b15918

TMP: 10k citations

7f61aa4

TMP: announcements

26ff776

add back Python helpers

066f8af

Some other cleanup

6c40ce8

Attempt to fix circlci

47c1761

Update .gitignore

073c927

Use theme in new home

e7e55f9

Remove now redundant subs

edab2ba

fix artifact path

f18d04b

Cadair force-pushed the unified-theme-rewrite branch from b122766 to f18d04b Compare February 5, 2026 16:07

Cadair added 4 commits February 5, 2026 16:17

Drop circleCI and use GitHub actions for checks

c57fc94

Fix many linkcheck errors, but not them all

d108dcb

use intersphinx for references to docs.astropy.org

Add a workflow for publishing to github pages

f75fd07

Merge remote-tracking branch 'upstream/main' into unified-theme-rewrite

2894e76

Cadair added 2 commits April 8, 2026 10:28

Address linkcheck errors

5e25664

bump to release theme

36733a0

eteq reviewed May 6, 2026

View reviewed changes

eteq merged commit 6962b36 into astropy:main May 6, 2026
6 of 8 checks passed

eteq mentioned this pull request May 6, 2026

Remove circleci webhook #706

Open

This was referenced May 11, 2026

Credits page is missing #709

Open

Browser refuses to show RTD PR preview #713

Open

Fix links #711

Merged

+              sphinx
+              myst-parser
+              sphinx-design
+              astropy-sphinx-theme@git+https://github.com/Cadair/astropy-sphinx-theme@sunpy-astropy-theme

+              // Quirk to note: the jQuery.getJSON function fails if you open this locally
+              // with Chrome, because Chrome thinks local JSON files are unsafe for some
+              // reason.  Use basically any other modern browser, or it works fine if its
+              // actually on the web server even with chrome.

+              sphinx
+              myst-parser
+              sphinx-design
+              astropy-sphinx-theme@git+https://github.com/Cadair/astropy-sphinx-theme@sunpy-astropy-theme

Rewrite to a sphinx project and use unified theme by Cadair · Pull Request #698 · astropy/astropy.github.com · GitHub

Uh oh!

Uh oh!

Rewrite to a sphinx project and use unified theme - #698

Rewrite to a sphinx project and use unified theme#698eteq merged 25 commits intoastropy:mainastropy/astropy.github.com:mainfrom Cadair:unified-theme-rewriteCadair/astropy.github.com:unified-theme-rewriteCopy head branch name to clipboard

Conversation

Cadair commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Cadair commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

pllim commented Jan 20, 2026

Uh oh!

Cadair commented Jan 21, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pllim commented Jan 21, 2026

Uh oh!

Cadair commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hamogu commented Jan 21, 2026

Uh oh!

hamogu left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pllim commented Jan 22, 2026

Uh oh!

Cadair commented Jan 27, 2026

Uh oh!

pllim commented Jan 27, 2026

Uh oh!

Cadair commented Jan 27, 2026

Uh oh!

pllim commented Jan 27, 2026

Uh oh!

pllim commented Jan 27, 2026

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pllim left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Cadair commented Jan 27, 2026

Uh oh!

pllim left a comment

Choose a reason for hiding this comment

Uh oh!

Cadair commented Jan 27, 2026

Uh oh!

Cadair commented Apr 7, 2026

Uh oh!

eteq left a comment

Choose a reason for hiding this comment

Uh oh!

eteq commented May 6, 2026

Uh oh!

eteq commented May 6, 2026

Uh oh!

Uh oh!

eteq commented May 6, 2026

Uh oh!

Reviewers

Assignees

Rewrite to a sphinx project and use unified theme#698
eteq merged 25 commits into
astropy:mainfrom
Cadair:unified-theme-rewrite

Cadair commented Jan 19, 2026 •
edited
Loading

Cadair commented Jan 19, 2026 •
edited
Loading

Cadair commented Jan 21, 2026 •
edited
Loading