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 malformed pagination urls attempt #2 #36

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

Conversation

gigago
Copy link
Contributor

@gigago gigago commented Nov 7, 2018

The previous 'unfix' of pagination urls unfixed my application :-). Since the resulting urls are in fact malformed when base_url contains a scheme id and some browsers don't like this, let's try again to solve this in a different way by appending the filter |replace({':/':'://'}) to each url. Supposedly, this will only affect the scheme part of the url, if any.

@juffalow
Copy link

Your suggestion with |replace({':/':'://'}) fixed my problem. While this is not merged, the only solution is to create the pagination.html.twig file in my theme.

@gigago
Copy link
Contributor Author

gigago commented Jan 18, 2019 via email

@juffalow
Copy link

@gigago it is still not good :( Now the pages are with three /. Browser will fix it so it is working but when you check the source code, it is wrong. https://juffalow.com and the pagination on the bottom is https:///.

@juffalow
Copy link

Btw, for me, the previous version is working cfb8064.

@gigago
Copy link
Contributor Author

gigago commented Jan 22, 2019 via email

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

Successfully merging this pull request may close these issues.

None yet

2 participants