8000 Fix editor scroll position by kntsoriano · Pull Request #1306 · exercism/website · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix editor scroll position #1306

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

Closed
wants to merge 1 commit into from
Closed

Fix editor scroll position #1306

wants to merge 1 commit into from

Conversation

kntsoriano
Copy link
Contributor
@kntsoriano kntsoriano commented Jul 26, 2021

exercism/v3-beta#143

This is a known error for Turbolinks, and I copied the solution here: turbolinks/turbolinks#181 (comment)

It seems to me that turbolinks/turbolinks#420 is the PR for this fix, but it won't seem to get merged anytime soon. Should we look into using Hotwire now?

@iHiD
Copy link
Member
iHiD commented Jul 26, 2021

Hmmm, it looks like this broke the tests.

@iHiD
Copy link
Member
iHiD commented Jul 26, 2021

@kntsoriano I think we want Turbo, not Hotwire (Hotwire also installs Stimulus, which we don't want to use at least for now).

Spend a day working on replacing Turbolinks with Turbo. See whether it works or not. If it looks like it'll be too painful, we'll just abort, but if we can do it, that'd be good 🙂

@iHiD
Copy link
Member
iHiD commented Aug 5, 2021

@kntsoriano This would be a good one for you to look at next week while I'm away.

@iHiD
Copy link
Member
iHiD commented Aug 17, 2021

@kntsoriano Can we close this now?

@kntsoriano kntsoriano closed this Aug 17, 2021
@kntsoriano kntsoriano deleted the scroll-position branch August 17, 2021 21:44
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.

2 participants
0