Skip to content

DOC: restore navigation documentation #25926

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

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

tacaswell
Copy link
Member

PR summary

In bb8058a via #4779 I accidentally deleted the (illustrated) section on what the buttons on the default toolbar do. I suspect that this was a bad rebase as I do not think I would have intentionally deleted this content.

This commit:

  • restores the content as-was: git checkout b49973a doc/users/navigation_toolbar.rst
  • fixes the paths to images (the files have moved around)
  • copy the navigation content into interactive.rst and re-remove navigation_toolbar.rst

Partially addresses #25266.

Overlaps with #25925, sorry about that @satisf1ed . A benefit of doing it "the old way" as restored here is that the images we are pulling are the exact files we use to generate the toolbars so they will never get out of sync.

PR checklist

In bb8058a via matplotlib#4779 I accidentally deleted
the (illustrated) section on what the buttons on the default toolbar do.  I
suspect that this was a bad rebase as I do not think I would have intentionally
deleted this content.

This commit:

 - restores the content as-was:
    git checkout b49973a doc/users/navigation_toolbar.rst
 - fixes the paths to images (the files have moved around)
 - copy the navigation content into interactive.rst and re-remove
   navigation_toolbar.rst

Partially addresses matplotlib#25266.
@ksunden ksunden merged commit 030bf3a into matplotlib:main Jun 14, 2023
@tacaswell tacaswell deleted the doc/restore_navigation_docs branch January 4, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants