Skip to content

Remove unused related images from docs#712

Merged
rahulporuri merged 5 commits into
masterfrom
cln/remove-shell-images-from-docs
Apr 22, 2021
Merged

Remove unused related images from docs#712
rahulporuri merged 5 commits into
masterfrom
cln/remove-shell-images-from-docs

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

@rahulporuri rahulporuri commented Apr 22, 2021

The doc pages which used these images waere removed in the PRs #668 and #669 but we forgot to remove the images in the PRs. This PR also removes a copy of simple_line.png and instead just makes use of the one copy everywhere. This PR also removes an xcf file from the docs and im not even sure who/what uses the file.

Poruri Sai Rahul added 4 commits April 22, 2021 10:32
the doc page which used these images was removed in the PR #669
but we forgot to remove the images in the PR

	deleted:    docs/source/user_manual/images/ipython_basic.png
	deleted:    docs/source/user_manual/images/ipython_kde.png
	deleted:    docs/source/user_manual/images/ipython_multi.png
the quickstart page got removed in #668

	deleted:    docs/source/images/lines.png
	deleted:    docs/source/images/lines_final.png
	deleted:    docs/source/images/mylineplot.png
	deleted:    docs/source/images/simple_line.png
	deleted:    docs/source/user_manual/images/simple_line.png
and just use the one image everywhere needed in docs

	modified:   docs/source/user_manual/annotated_examples.rst
	deleted:    docs/source/user_manual/example_images/simple_line.png
but it isnt used in the docs so lets remove it

	deleted:    docs/source/images/chaco_show.xcf
@rahulporuri rahulporuri changed the title Remove "chaco.shell" related images from docs Remove unused related images from docs Apr 22, 2021
and use the one copy everywhere

	new file:   docs/source/images/simple_line.png
	modified:   docs/source/user_manual/annotated_examples.rst
	modified:   docs/source/user_manual/chaco_tutorial.rst
Copy link
Copy Markdown
Contributor

@aaronayres35 aaronayres35 left a comment

Choose a reason for hiding this comment

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

LGTM

@rahulporuri rahulporuri merged commit ae6248b into master Apr 22, 2021
@rahulporuri rahulporuri deleted the cln/remove-shell-images-from-docs branch April 22, 2021 13:02
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