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

Experimental support of Turbolinks 3 #6

Closed
wants to merge 9 commits into from

Conversation

tortuetorche
Copy link
Contributor

The goal of this pull request is just to show how we can support Turbolinks 3 in Symfony/Laravel.
I'm open to any suggestions and/or modifications!

Warning: This is an alpha version, it isn't production-ready yet.

Bonus: You can play with this online demo (built with Laravel).
And you can grab the source code of this demo here.

Should fix #4

Tortue Torche and others added 3 commits July 22, 2015 14:55
@tortuetorche
Copy link
Contributor Author

For the Laravel users: you can found here the Turbolinks 3 stuff.
I plan to merge it in the Frenzy Turbolinks repository later.

@carlalexander
Copy link
Contributor

Thanks for that! I'll leave this here for when the new version is released.

@carlalexander
Copy link
Contributor

In light of some of the recent information about Turbolinks development, this PR is probably not going to get merged as it stands. For now, I'll leave it open for anyone using it.

I plan on working on migrating the component to v5 when they start merging the work they're doing at Basecamp upstream. Until then, the recommendation is to stick with the current stable version,

@tortuetorche
Copy link
Contributor Author

Hi @carlalexander,

I think the PHP code for Turbolinks 5 will not be so much different than this one for Turbolinks 3.
So, I will push a new pull request for Turbolinks 5, when it will be ready.

Cheers,
Tortue Torche.

@carlalexander
Copy link
Contributor

Did you ever get anything working for Turbolinks 5?

@tortuetorche
Copy link
Contributor Author

Hi @carlalexander,

Yes, I did.

I can send a new pull request for the Turbolinks 5 compatibility, if you want.

Cheers,
Tortue Torche

@carlalexander
Copy link
Contributor

I'll close this one then. 😃

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.

Add support for partial page replacement strategies with Turbograft
2 participants