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

DOC: Simplify pandas theme footer #52149

Closed
wants to merge 1 commit into from

Conversation

natmokval
Copy link
Contributor

@natmokval natmokval commented Mar 23, 2023

This pr is related to issue #51536. The next changes were made:

  1. Changed a version of pydata-sphinx-theme in environment.yml from less than 0.11 to 0.13.0
  2. Updated my environment by running mamba env update -n pandas-dev --file environment.yml
  3. In conf.py I changed html_theme_options. Now "footer_start" is used instead of "footer_items”
  4. In doc/source/_static/css/pandas.css where we override some aspects of pydata-sphinx-theme I removed card-footer
  5. After cleaning and rebuilding documentation there is no footer in our html.

@natmokval natmokval changed the title change version of pydata-sphinx-theme and html_theme_options DOC: Simplify pandas theme footer Mar 23, 2023
@mroeschke mroeschke added Docs Web pandas website labels Mar 24, 2023
Copy link
Member

@datapythonista datapythonista left a comment

Choose a reason for hiding this comment

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

Thanks for working on this @natmokval, added some comments that I think should help.

@@ -36,17 +36,6 @@ table {
padding: 2.5rem 0rem 0.5rem 0rem;
}

.intro-card .card-footer {
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we should modify the css for this. Am I missing something? I think you probably want to revert this deletion.

@@ -240,7 +240,7 @@

html_theme_options = {
"external_links": [],
"footer_items": ["pandas_footer", "sphinx-version"],
"footer_start": ["pandas_footer", "sphinx-version"],
"github_url": "https://github.com/pandas-dev/pandas",
"twitter_url": "https://twitter.com/pandas_dev",
"google_analytics_id": "UA-27880019-2",
Copy link
Member

Choose a reason for hiding this comment

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

For the error in the CI seems like this config has veen removed: WARNING: unsupported theme option 'google_analytics_id' given

Can you check on the pydata theme docs which is the new way to do this please?

@@ -87,7 +87,7 @@ dependencies:
- gitdb
- natsort # DataFrame.sort_values doctest
- numpydoc
- pydata-sphinx-theme<0.11
- pydata-sphinx-theme>=0.13.0
Copy link
Member

Choose a reason for hiding this comment

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

I think it's better to simply remove the version here, it should pick the last one anyway.

@@ -62,7 +62,7 @@ gitpython
gitdb
natsort
numpydoc
pydata-sphinx-theme<0.11
pydata-sphinx-theme>=0.13.0
Copy link
Member

Choose a reason for hiding this comment

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

Same

@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2023

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

@github-actions github-actions bot added the Stale label May 1, 2023
@datapythonista
Copy link
Member

@natmokval do you still want to work on this?

@mroeschke
Copy link
Member

Thanks for the pull request, but it appears to have gone stale. If interested in continuing, please merge in the main branch, address any review comments and/or failing tests, and we can reopen.

@mroeschke mroeschke closed this May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Stale Web pandas website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants