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

Use Resource#url to honour directory indexes #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

assimovt
Copy link

@assimovt assimovt commented Sep 4, 2016

Resource#path does not work well with Middleman Pretty URLs,
see https://middlemanapp.com/advanced/pretty_urls

Using Resource#url fixes the problem as it follows the app configs.

Resource#path does not work well with Middleman Pretty URLs,
see https://middlemanapp.com/advanced/pretty_urls

Using Resource#url fixes the problem as it follows the app configs.
assimovt pushed a commit to Dockbit/docs that referenced this pull request Sep 4, 2016
Until marnen/middleman-breadcrumbs#12 is merged
used this custom fork to fix the pretty-url links in breadcrumbs
@marnen
Copy link
Owner

marnen commented Sep 4, 2016

#9 might be related to this.

@assimovt
Copy link
Author

assimovt commented Sep 4, 2016

Resource#url seems to be handling it already.

@marnen
Copy link
Owner

marnen commented Sep 8, 2016

This will need tests before I can merge it. Also, what is the problem with path?

@marnen
Copy link
Owner

marnen commented Dec 5, 2016

Hmm...I see that this has tests, which I somehow missed last time I looked. :P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants