Skip to content

Commit

Permalink
Regression for #2332 (#2385)
Browse files Browse the repository at this point in the history
* Regression for #2332

There's already a `relative_url` in place, shouldn't stack up another

* Update CHANGELOG and history
  • Loading branch information
iBug authored Jan 29, 2020
1 parent 636ea3e commit 1799ddd
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 6 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
## Unreleased

### Bug Fixes

- Fix repeated site base path for masthead logo. [#2385](https://github.com/mmistakes/minimal-mistakes/pull/2385)

## [4.18.1](https://github.com/mmistakes/minimal-mistakes/releases/tag/4.18.1)

### Bug Fixes
Expand Down
6 changes: 1 addition & 5 deletions _includes/masthead.html
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
{% if site.logo contains "://" %}
{% capture logo_path %}{{ site.logo }}{% endcapture %}
{% else %}
{% capture logo_path %}{{ site.logo | relative_url }}{% endcapture %}
{% endif %}
{% capture logo_path %}{{ site.logo }}{% endcapture %}

<div class="masthead">
<div class="masthead__inner-wrap">
Expand Down
8 changes: 7 additions & 1 deletion docs/_docs/18-history.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,16 @@ permalink: /docs/history/
excerpt: "Change log of enhancements and bug fixes made to the theme."
sidebar:
nav: docs
last_modified_at: 2020-01-25T11:15:45-05:00
last_modified_at: 2020-01-29T17:42:20+08:00
toc: false
---

## Unreleased

### Bug Fixes

- Fix repeated site base path for masthead logo. [#2385](https://github.com/mmistakes/minimal-mistakes/pull/2385)

## [4.18.1](https://github.com/mmistakes/minimal-mistakes/releases/tag/4.18.1)

### Bug Fixes
Expand Down

0 comments on commit 1799ddd

Please sign in to comment.