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

Fix #4082: Allow users to use .htm and .xhtml (XHTML5.) #4160

Merged
merged 1 commit into from Nov 25, 2015

Conversation

envygeeks
Copy link
Contributor

No description provided.

@parkr
Copy link
Member

parkr commented Nov 19, 2015

Should this also be applied to documents?

@parkr parkr added the fix label Nov 19, 2015
@parkr parkr added this to the 3.1 milestone Nov 19, 2015
@parkr
Copy link
Member

parkr commented Nov 19, 2015

Should this also be applied to documents?

@envygeeks ^^

@envygeeks
Copy link
Contributor Author

@parkr I tried but I couldn't figure out where (but that's probably because I was dealing with a storm of bugs in our stack yesterday so I wasn't paying attention.) I'll play with it today and make sure documents work as expected and update and ping if I need to!

@parkr
Copy link
Member

parkr commented Nov 19, 2015

Thanks :)

@parkr
Copy link
Member

parkr commented Nov 23, 2015

Document code:

def destination(base_directory)
dest = site.in_dest_dir(base_directory)
path = site.in_dest_dir(dest, URL.unescape_path(url))
path = File.join(path, "index.html") if url.end_with?("/")
path << output_ext unless path.end_with?(output_ext)
path
end

@envygeeks
Copy link
Contributor Author

On it, I forgot about this when I was doing stuff. I'll get it fixed up.

parkr added a commit that referenced this pull request Nov 25, 2015
@parkr parkr merged commit ac1b2b8 into master Nov 25, 2015
@parkr parkr deleted the fix-html-htm-xhtml-pages branch November 25, 2015 00:19
parkr added a commit that referenced this pull request Nov 25, 2015
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants