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

Replace sphinx-panels for sphinx-design #329

Merged
merged 4 commits into from
Jul 6, 2022
Merged

Replace sphinx-panels for sphinx-design #329

merged 4 commits into from
Jul 6, 2022

Conversation

santisoler
Copy link
Member

Refactor the home page of Harmonica docs to use sphinx-design grid and cards.
Add some icons to the cards and redesign the header to be actual text instead
of a png banner. Add cards for each geometric body in Forward Modelling index
page and reorganize its content.

Relevant issues/PRs:

Inspired by fatiando/boule#112

Refactor the home page of Harmonica docs to use sphinx-design grid and
cards. Add some icons to the cards and redesign the header to be actual
text instead of a png banner.
@leouieda
Copy link
Member

Looks great, @santisoler!

As a side note, the install seems to be failing because netcdf4 doesn't have compiled wheels on python 3.7. Is it necessary for the tests?

@leouieda leouieda closed this Jun 29, 2022
@leouieda leouieda reopened this Jun 29, 2022
@leouieda
Copy link
Member

Sorry, clicked the close button by mistake.

@santisoler
Copy link
Member Author

As a side note, the install seems to be failing because netcdf4 doesn't have compiled wheels on python 3.7. Is it necessary for the tests?

Good question and excellent timing. I added netcdf4 as requirement for testing because I created a sample 2d grid for the FFT tests. But with #328 we don't need it anymore.

@santisoler santisoler merged commit a276a2c into main Jul 6, 2022
@santisoler santisoler deleted the sphinx-design branch July 6, 2022 18:35
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.

2 participants