Skip to content
This repository has been archived by the owner on May 12, 2020. It is now read-only.

Fix turbolinks issue which caused infinite scrollbar ond other UI loa… #45

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AjayBarot
Copy link
Contributor

@AjayBarot AjayBarot commented Mar 16, 2017

@kenny-hibino : Please review the changes. I tested overall application but still let me know If you inspect something unusual.
#35

@hibiken
Copy link
Owner

hibiken commented Mar 22, 2017

@AjayBarot I tested locally and I think animation is not working with this change 😢

Fix animation issue caused by turbolinks gem version changes.
2. disable turbolinks caching on each layout to solve back button issue.
@AjayBarot
Copy link
Contributor Author

@kenny-hibino : thanks for testing this changes into local. I have made some changes to cope up with back button issue and animation issue. Please have a look at this changes. https://github.com/kenny-hibino/stories/pull/45/files. and let me know your feedback after testing of this latest changes. Thank you pal.

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

Successfully merging this pull request may close these issues.

None yet

2 participants