-
Notifications
You must be signed in to change notification settings - Fork 25.6k
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
Update masthead.html #2332
Update masthead.html #2332
Conversation
fix image path in masthead for relative url
Co-Authored-By: iBug ♦ <git@ibugone.com>
This change breaks the site-logo for jekyll 4.0 with a baseurl. With this change I get ERROR '/~ayoung48/~ayoung48/assets/images/logo88x88.png' not found. If I undo this change, so that the line is {% capture logo_path %}{{ site.logo }}{% endcapture %} this issue goes away. |
@Geekdude Hi, can you please share the exact value for your |
@iBug Hi, here are the logo and baseurl settings from
The issue could be related to changes made in jekyll 4.0.
|
There's already a `relative_url` in place, shouldn't stack up another
@Geekdude Don't think it's a Jekyll 4.0 issue. That quote above is in reference to when you use Jekyll's I think you may be misusing For example. If I had a domain of
Which would give me URLs like With your settings you'd get |
@mmistakes I host my website on my university's servers. They use |
@Geekdude OK. That makes more sense then. |
@iBug Cool, thank you for finding and fixing the issue. |
* Update masthead.html fix image path in masthead for relative url * Update _includes/masthead.html Co-Authored-By: iBug ♦ <git@ibugone.com>
* Update masthead.html fix image path in masthead for relative url * Update _includes/masthead.html Co-Authored-By: iBug ♦ <git@ibugone.com>
* Regression for mmistakes#2332 There's already a `relative_url` in place, shouldn't stack up another * Update CHANGELOG and history
* Regression for mmistakes#2332 There's already a `relative_url` in place, shouldn't stack up another * Update CHANGELOG and history
* Update masthead.html fix image path in masthead for relative url * Update _includes/masthead.html Co-Authored-By: iBug ♦ <git@ibugone.com>
* Update masthead.html fix image path in masthead for relative url * Update _includes/masthead.html Co-Authored-By: iBug ♦ <git@ibugone.com>
* Regression for mmistakes#2332 There's already a `relative_url` in place, shouldn't stack up another * Update CHANGELOG and history
* Update masthead.html fix image path in masthead for relative url * Update _includes/masthead.html Co-Authored-By: iBug ♦ <git@ibugone.com>
* Regression for mmistakes#2332 There's already a `relative_url` in place, shouldn't stack up another * Update CHANGELOG and history
fix image path in masthead for relative url
This is a bug fix.
Summary
Absolute/relative path check {% if site.logo contains "://" %} for logo image must prepend the baseurl in case of relative path.
Context
If you specify 'logo: "/assets/images/logoexample.png' in _config.yml, logoexample.png will not be shown in masthead unless absolute path is used.