Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Fixing broken image links#481

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
shrayasr wants to merge2 commits intoclojure-cookbook:master
base:master
Choose a base branch
Loading
fromshrayasr:fix-image

Conversation

shrayasr
Copy link

The images for4-25_making-a-window.asciidoc are referenced from a saidimages/ folder that is non existent. These are now fixed by using images from the same directory.

One small problem would be that in this example

(deff (frame:title"Lyrical Clojure"))(show! f)

We say that the title is set to "Lyrical Clojure" but the image shows "Cooki...". Doesn't makeTOO much of a difference for sure, just a nitpick.

Also thanks so much for this. I've learnt a lot over the past week.

Cheers. 🍺

@shrayasr
Copy link
Author

Note: Since thefix-image branch doesn't exist on your side, the images aren't being displayed in the rich diff. You can see it working onmy fork

@shrayasr
Copy link
Author

Update: Noticed a broken link in another doc as well, fixed it.

@qtrfeast
Copy link
Member

Theimages/folder is actually an artifact of O'Reilly's manuscript preparation process. Ideally, we'd fix the broken images by getting the links to theimages/ folder to work.

@shrayasr
Copy link
Author

Oh I see. Should I make an images folder and move the files there?
On Jul 24, 2015 12:32 AM, "Ryan Neufeld"notifications@github.com wrote:

The images/folder is actually an artifact of O'Reilly's manuscript
preparation process. Ideally, we'd fix the broken images by getting the
links to the images/ folder to work.

—
Reply to this email directly or view it on GitHub
#481 (comment)
.

@qtrfeast
Copy link
Member

They should already be present in/images

@shrayasr
Copy link
Author

Ah they are. Sorry about this. Seems like I didn't study the structure enough. Please feel free to close this PR.

However I have one subsequent question. For anyone trying to read this online, its hard to understand and gives the impression of broken links (like it did for me). Do you think we can fix this in anyway?

Also, thanks for the clarification. 👍. Appreciate it.

@qtrfeast
Copy link
Member

There must be some way to fix the images online. I was hoping there was some way to correct the broken path in the links to point at the correct place.

@tirkarthitirkarthi mentioned this pull requestSep 1, 2017
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@shrayasr@qtrfeast
[8]ページ先頭
©2009-2025 Movatter.jp