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

Page toc (right sidebar) content and styling #12

Closed
jorisvandenbossche opened this issue Sep 9, 2019 · 5 comments
Closed

Page toc (right sidebar) content and styling #12

jorisvandenbossche opened this issue Sep 9, 2019 · 5 comments

Comments

@jorisvandenbossche
Copy link
Member

Apart from #8 on collapsing the sections that you are not viewing, there are some other things we might want to do for the right sidebar.

We can use this issue to gather some ideas.

@jorisvandenbossche jorisvandenbossche changed the title Page toc (right sidebar) styling Page toc (right sidebar) content and styling Sep 9, 2019
@jorisvandenbossche
Copy link
Member Author

Some other things we might want to put in the right sidebar:

  • A "On this page" or "Table of contents" title, to make it clearer what this navigation is exactly
  • A "Edit this page" link

Example from docker (see eg https://docs.docker.com/engine/reference/commandline/dockerd/), although here it is rather crowded with additional things:

image

Jupyter book also uses a "On this page" in the right sidebar (eg https://jupyterbook.org/intro):

image

@mangecoeur
Copy link

Adding 'On This Page' would definitely help make it clearer - although the Jupyter example has its own problems (namely that there are two unrelated navigation hierarchies in the left and right panes - more of a navigation bush than a tree >_< )

Probably worth listing (at least for reference) the pros (and cons) for the left/right split navigation vs the collapsing tree navigation of the old docs. My own take:

  • now that the 'top level' nav is in the header there would be more space in the left pane for a classic tree nav, which would be simpler and always shown
  • by using the right nav, more sub-levels of the navigation tree can be show, which lets the docs grow without becoming impossible to navigate because you can always jump to sub sections.

@jorisvandenbossche
Copy link
Member Author

jorisvandenbossche commented Oct 30, 2019

Thanks for your considerate thoughts!

I think it is clear that it is useful to get to the subsections of a page through the navigation (at some point we briefly disabled this: pandas-dev/pandas#25131). But the question is where to put them.

Before we had a single (left sidebar) navigation structure. In this theme we opted for 3 (top navbar, left sidebar, right sidebar). It's certainly true limiting to 2 structures would already be an improvement over the old situation IMO.
(and it's also subjective, personally I like the 3 navigation elements, but I know others of the team find it "too much". It maybe also depends on the way how you use the docs)

Reasons that I prefer to have the "within page toc" and sidebar separate:

  • When showing the page toc also in the navigation tree on the left, this quickly bcomes a long tree for pages that are somewhat longer. In the old theme, on some pages, this means that the full tree doesn't fit on my screen (eg on https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html). And removing the top-level (what is now in the top navbar) only improves this slightly, and for the example above, it would still not fit on my screen (certainly if you make the font a bit larger, which it is in the new theme).

  • I would like to split some of our very long pages into multiple pages. While that makes the previous argument potentially moot, I would like to do this by introducing an extra level in the navigation. Eg now, you have "User Guide" -> "IO tools" (2 levels). I would like to split the big IO page into multiple pages by introducing a third level. This means that the left sidebar would also already show 2 levels (see eg https://dev.pandas.io/docs/getting_started/comparison/comparison_with_r.html, where we already have this). In such a case, also showing the within-page toc in this left sidebar, would make it very nested (certainly if you want to show more than 1 additional level of the page).

@jpmckinney
Copy link
Contributor

"Edit this page" and "On this page" are implemented, e.g. https://pydata-sphinx-theme.readthedocs.io/en/latest/demo/demo.html

Should this issue be closed?

@choldgraf
Copy link
Collaborator

Yeah I think so, we can open up new issues for specific ideas to track more easily I think

drammock added a commit that referenced this issue Oct 20, 2023
* docs: add the full option reference in the docs

* trigger CI

* refactor: remove line start

* update to latest main (#12)

* chore(i18n) catalan (#1488)

i18n: Translate sphinx.po in ca

100% translated source file: 'sphinx.po'
on 'ca'.

Co-authored-by: transifex-integration[bot] <43880903+transifex-integration[bot]@users.noreply.github.com>

* Build(deps): Bump postcss and css-loader (#1494)

Bumps [postcss](https://github.com/postcss/postcss) to 8.4.31 and updates ancestor dependency [css-loader](https://github.com/webpack-contrib/css-loader). These dependencies need to be updated together.


Updates `postcss` from 8.4.21 to 8.4.31
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.4.21...8.4.31)

Updates `css-loader` from 3.6.0 to 6.8.1
- [Release notes](https://github.com/webpack-contrib/css-loader/releases)
- [Changelog](https://github.com/webpack-contrib/css-loader/blob/master/CHANGELOG.md)
- [Commits](webpack-contrib/css-loader@v3.6.0...v6.8.1)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: indirect
- dependency-name: css-loader
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Revert "Build(deps): Bump postcss and css-loader" (#1509)

Revert "Build(deps): Bump postcss and css-loader (#1494)"

This reverts commit 185a37a.

* Update pst docs buttons (#1502)

* call them button-links

* copy edit

* docs: link back to GitHub from PyPI metadata (#1504)

This will add a "Source" link in the PyPI page.

* navigation_with_keys = False (#1503)

* navigation_with_keys = False

* None -> False

* Apply suggestions from code review

---------

Co-authored-by: Daniel McCloy <[email protected]>

* fix: convert "stable" to actual version number (#1512)

* convert "stable" to actual version number

* fix tests re: navigation_with_keys

* try bumping autoapi

* refactor: use nbsphinx as the default execution lib (#1482)

* refactor: use nbsphinx as the default execution lib

* add nbstripout to the pre-commits'

* add pandoc to the readthedocs deps

* refactor: clean the notebook

* move the example to the correct folder

* fix: solve link issue

* install pandoc in the test environment

* fix: display of large table in executed cells

* avoid Userwarnings from matplotlib

* hide the matplotlib wrning management cell

* Update readthedocs.yml

* build: use pandoc_binary to install pandoc

* docs: add reference to pandoc in the setup

* update docs

* remove pypandoc_binary

* Update pyproject.toml

Co-authored-by: gabalafou <[email protected]>

* ci: use back setup-pandoc

* Trigger CI build

---------

Co-authored-by: Gabriel Fouasnon <[email protected]>

* BUG - Clear alt_text in conf.py (#1471)

* comment out alt_text in conf.py

* set alt_text to empty string

* remove alt_text from conf.py

* fix: use 12rambau fork until it's merged with nikeee repo (#1517)

* deps: drop support for Sphinx 5 (#1516)

* remove ref to myst-nb

* update minimal supported version of sphinx

* Fix: (webpack.config.js) css-loader API change (#1508)

* Fix: (webpack.config.js) css-loader API change

The build was broken in
<https://github.com/pydata/pydata-sphinx-theme/commit/185a37aa36820f77bffa4c87a772092e9e7cc380>/<https://github.com/pydata/pydata-sphinx-theme/pull/1494>.

This change fixes the build, and it seems to be in accordance with the
current API as described at <https://github.com/webpack-contrib/css-loader/blob/c6f36cf91ac61743a70e81cfb077faa0f8730ebe/README.md#boolean>.

Closes <#1507>.

* dedup

* restore version bump

---------

Co-authored-by: Daniel McCloy <[email protected]>

* Fix duplicate HTML IDs (#1425)

* Fix duplicate HTML IDs

* fix tests

* Do not animate the version admonitions colors. (#1424)

Otherwise a delay has to be added to the accessibility color
contrast checks, to wait for the colors to fully transition.

* BUG - Remove redundant ARIA in breadcrumb navigation (#1426)

* style(i18n): French Typo fixed (#1430)

* Add the ability to add a center section to the footer (#1432)

* Add a center section for the footer

* Add docs for footer_center

* Add a test site for the center footer

* test it in our own docs

* remove new test site

* add footer test

---------

Co-authored-by: Daniel McCloy <[email protected]>

* Build(deps): Bump actions/checkout from 3 to 4 (#1433)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add dropdown_text argument to generate_header_nav_html (#1423)

* Add dropdown_text argument to generate_header_nav_html

* Add a test, fix typo in theme.conf and remove header_dropdown_text from docs/conf.py

* fixed?

---------

Co-authored-by: Daniel McCloy <[email protected]>

* fix: rollback ref and Id changes (#1438)

* bump: version 0.13.3 → 0.14.0  (#1440)

* bump version

* update version switcher

* back to dev

* fix: change the z-index of the dropdown (#1442)

In order to be on top of the primary sidebar on small screens.

* fix: set the same background for dark/light (#1443)

* fix: set the same background for dark/light
et the same background color for all state of the search field. It is currently only applied when hovered

* fix: wrong css selector

* Update src/pydata_sphinx_theme/assets/styles/components/_search.scss

* Update src/pydata_sphinx_theme/assets/styles/components/_search.scss

* Fix duplicate HTML IDs

* fix tests

* unique_html_id

* backwards-compat generate_header_nav_html

* feedback review

* update fixture

* ughhhh...caching

* code cleanup

* fix test snapshot

* put comment inside def

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Denis Bitouzé <[email protected]>
Co-authored-by: Stuart Mumford <[email protected]>
Co-authored-by: Daniel McCloy <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Alenka Frim <[email protected]>
Co-authored-by: Rambaud Pierrick <[email protected]>

* chore: build the devcontainer automatically in codespace (#1483)

* chore: build the devcontainer automaticallyin codespace

* refactor: lint

* add pandoc to the environment

* Fix font color in search input box (#1524)

* Fix color

* Use --pst-color-text-base

* docs: add DecentralChain (#1528)

Co-authored-by: jourlez <[email protected]>

* Updates for file src/pydata_sphinx_theme/locale/en/LC_MESSAGES/sphinx.po in ru [Manual Sync] (#1527)

i18n: Translate sphinx.po in ru [Manual Sync]

96% of minimum 20% translated source file: 'sphinx.po'
on 'ru'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format

Co-authored-by: transifex-integration[bot] <43880903+transifex-integration[bot]@users.noreply.github.com>

* ignore transient errors in windows build CI (#1520)

* use warning list

* clean up notebook

* refactor to pass on all platforms?

* simplify

* fix logic

* iterate backwards

* fix plaform detection? also don't log unnecessarily�[H

* ignore empty string warnings

* remove notebook metawarning

* Revert "remove notebook metawarning"

This reverts commit 42f4672.

* try again

* debug the mysterious empty warning

* escape color codes

* import

* triage by intermittency, not by platform; better var names

* simplify

* fix list.remove

* undo what I broke

* Update tests/utils/check_warnings.py

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: transifex-integration[bot] <43880903+transifex-integration[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel McCloy <[email protected]>
Co-authored-by: gabalafou <[email protected]>
Co-authored-by: Ned Batchelder <[email protected]>
Co-authored-by: Adam Porter <[email protected]>
Co-authored-by: Denis Bitouzé <[email protected]>
Co-authored-by: Stuart Mumford <[email protected]>
Co-authored-by: Alenka Frim <[email protected]>
Co-authored-by: Harutaka Kawamura <[email protected]>
Co-authored-by: jourlez <[email protected]>

* fix: use internal reference

* fix: last chance to make the internal reference to work

* fix crossref?

* fix crossref

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: transifex-integration[bot] <43880903+transifex-integration[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel McCloy <[email protected]>
Co-authored-by: gabalafou <[email protected]>
Co-authored-by: Ned Batchelder <[email protected]>
Co-authored-by: Adam Porter <[email protected]>
Co-authored-by: Denis Bitouzé <[email protected]>
Co-authored-by: Stuart Mumford <[email protected]>
Co-authored-by: Alenka Frim <[email protected]>
Co-authored-by: Harutaka Kawamura <[email protected]>
Co-authored-by: jourlez <[email protected]>
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

No branches or pull requests

4 participants