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

Adjusted image path in xcode.ko.md #816

Merged
merged 4 commits into from
Dec 13, 2023

Conversation

alexhojinpark
Copy link
Contributor

Reference: #815

I attempted to locate a path for images from ko/setup/xcode/xcode_setup_assets/install-dialog.webp, but couldn't find a solution(Image does not show up on my preview). I tested with VS Code default preview, and it seems adding ./ might work.

So, changing this:
![install dialog for the command line tools](xcode_setup_assets/install-dialog.webp)

to:
![install dialog for the command line tools](./xcode_setup_assets/install-dialog.webp)

Hope it works, Thank you.

@ofTheo
Copy link
Member

ofTheo commented Dec 12, 2023

I think this PR has a merge conflict @alexhojinpark

But if this doesn't work we might need to use absolute urls like:

![install dialog for the command line tools](https://openframeworks.cc/setup/xcode/xcode_setup_assets/install-dialog.webp)

@icq4ever
Copy link
Contributor

@alexhojinpark it's not shown on offline preview in repo because markdown files will be generate to static html file so leave image url as original will be better I guess. hope this helps.

@alexhojinpark
Copy link
Contributor Author

@ofTheo @icq4ever Thank you for the help. I've added absolute URLs. Should I create a new pull request for these changes? I'm not very familiar with the pull request process.

@icq4ever
Copy link
Contributor

icq4ever commented Dec 13, 2023 via email

@alexhojinpark
Copy link
Contributor Author

Thank you :)

@icq4ever
Copy link
Contributor

@alexhojinpark still there are conflicts.

@alexhojinpark
Copy link
Contributor Author

@icq4ever I'm assuming I have no permission to write or resolve conflicts, or I've done something wrong in Git commands while merging. I did a 'git pull' from master, switched to my branch, added, committed, and pushed to master.
Screenshot

@ofTheo ofTheo merged commit ef29f15 into openframeworks:master Dec 13, 2023
@ofTheo
Copy link
Member

ofTheo commented Dec 13, 2023

Not sure what the issue was, but I fixed it!
thanks @alexhojinpark

@alexhojinpark
Copy link
Contributor Author

@ofTheo @icq4ever Thank you so much for the help :) have a great day!

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.

3 participants