andrewbanchich / forty-jekyll-theme

A Jekyll version of the "Forty" theme by HTML5 UP.
https://andrewbanchich.github.io/forty-jekyll-theme/
Other
1.06k stars 2.17k forks source link

Remove the "/" remove extra / in url #132

Closed darkRaspberry closed 3 years ago

darkRaspberry commented 3 years ago

I can't replicate it on your demo webite.

Please verify it. As I'm getting extra "/" in my address bar if I'm using the header button to navigate in different page. If this doen't works for you close this pull.

How to reproduce.

Add a page

with dummy data and these headers

---
layout: (any)
title: (any)
nav-menu: true
description: Example 
image: assets/images/bannerDownload.png
---

Now when navigate to home by left top header button. It gives extra / in address bar like this..

http://example.com//

andrewbanchich commented 3 years ago

Could you provide a link to your example site so I can take a look?

darkRaspberry commented 3 years ago

@andrewbanchich Actually i have issue with license as I stashed the code base very much. And almost delete anything I don't want. I delete your credit on that files which i worked. I don't added my own (That's wrong though.) So i was concerned too much to share my fork. sorry.

I'm reading https://creativecommons.org/licenses/by/3.0/ still i didn't understand sorry. You may close this issue.