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

VSCode Sticky Scroll no longer works in ERB files after 0.7.10 #2314

Closed
brianclinkenbeard opened this issue Jul 16, 2024 · 6 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@brianclinkenbeard
Copy link

brianclinkenbeard commented Jul 16, 2024

Description

Reproduction steps

  1. Start Ruby LSP > 0.7.10 using vscode
  2. Open an ERB file
  3. Regression occurs where sticky scroll no longer appears

Example with a YAML file:
On 0.7.10:
image
On 0.7.12:
image

Likely related to the ERB code navigation change in #2235. I am using v0.17.7 of the language server.

@brianclinkenbeard brianclinkenbeard added the bug Something isn't working label Jul 16, 2024
@Ak-github-dev
Copy link

Hi @brianclinkenbeard,

I am new to this so probably correct if i am wrong , but as i see its working fine for me for the .erb files. I am using v0.7.13 of Ruby LSP in VSCode.

Example check: Check below screens:
image
when scrolled:
image
image
image

Check any logs you see in output panel once, Also try to add below in settings.json:

{
"ruby.useLanguageServer": true,
"ruby.languageServer.LaunchArgs": [
"--stdio",
"--verbose"
]
}
to log more detailed info

Please do ignore if this doesn't help or looks a noob solution.

Thanks

@andyw8
Copy link
Contributor

andyw8 commented Jul 18, 2024

Settings beginning with ruby. are not used by Ruby LSP – those are for a different extension.

@st0012
Copy link
Member

st0012 commented Jul 18, 2024

@brianclinkenbeard I can't reproduce the "fix" of downgrading extension. I do notice that I got sticky scroll back briefly when the extension restarts, like after downgrading or upgrading it.
Can you let me know if you still get the sticky scroll with the following steps:

  • Downgrade to v0.7.10 and restart the extension
  • Close the file
  • Open the file again

Additionally, can you check the language mark (example in screenshot) always indicates erb?

Screenshot 2024-07-18 at 21 14 25

And are you sure v0.7.10 is when the behaviour first changed?

@brianclinkenbeard
Copy link
Author

brianclinkenbeard commented Jul 20, 2024

@st0012 Yup, following those steps maintains no sticky scroll on the .yml.erb file I have open. The language mark in the bottom right always indicates erb (except when no version of Ruby LSP is installed, then it goes to Ruby). It seems that v0.7.11 is the first version when this occurs, and the sticky scroll goes away right after the LSP server starts.

@brianclinkenbeard
Copy link
Author

It looks like v0.7.14 fixes this issue, but I'm not sure why: I didn't disable the new rubyLsp.erbSupport setting, which defaults to true. 🤔

@vinistock
Copy link
Member

Since the issue seems to be gone, I'm closing this until we receive other reports. If you experience the issue again, let us know!

@vinistock vinistock closed this as not planned Won't fix, can't repro, duplicate, stale Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants