-
Notifications
You must be signed in to change notification settings - Fork 26
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
PR - Web icon fix #1724
PR - Web icon fix #1724
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really weird. The -r
meaning 'recursive' should have taken care of that. I'm kind of suspicious that something else is going on here. @Mornix , opinion?
Okay so 2 problems, the first being that the way I had my copy statement, the images folder was being copied into an images folder, hence Second problem being that my icon does not seem to be under version control, I will look into why. Once the image is pushed, I like the new way I have the |
|
Upon further inspection, it could possibly be due to different systems treating the first images has a Either way, I think my newer version is better as well. |
Weird that the image wasn't being tracked online, but on my system, it said it was. The new image seems to be appearing. |
@danscime's conclusions are generally correct. The way the |
Ok, I'll take this as a positive code review -- merged. |
Closes #1712. Closes #1633.
Website icon was not displaying due to some problems with the way the images folder was being copied.