-
Notifications
You must be signed in to change notification settings - Fork 252
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
Clicking a CSS path from inspector fails with remote environment #436
Comments
My debugging has led me to find a few things that, together, might be causing the issue here. Fundamentally, the So, unwinding a little more, the
And this is converted to
There are two problems with this string here, first is that the scheme is broken, it really should be This brings us to the question, why did the path separators flip, and what happened to one of the For that we need to dig into https://github.com/microsoft/vscode-edge-devtools/blob/main/src/utils.ts#L475, specifically the call to When we make this call,
But the return of the call to
Notice the So, what happened? Well, for that we dig into https://github.com/microsoft/vscode-chrome-debug-core/blob/main/src/utils.ts#L677-L685 and notice that it's using the I guess there needs to be a way to detect if it's a remote, and if it is, do path normalisation based on the remote file system semantics, or can you get around not doing path normalisation? |
Again, using the debugger, I edited the value that is assigned to The next problem is that using
I just so happened to have other files open in the editor at the time and if we have a look at the
Things of most interest are the So, how do we generate that? Using
The value of So by using the debugger to hack the value of |
I'd love to work with you folks to find a way to implement a proper solution to this, not just the really hacky one that I have for testing 🤣 |
Just tested this wit 1.3.1 and it looks like the changes from #467 have regressed (possibly in #475) Now when you click a link in the CSS inspector on WSL remote it pops an error dialog showing a message:
|
This has unfortunately gotten stale. Let me ping @vidorteg again to put this back on his radar. It'd be good to track this regression and work on a fix. |
This looks stale again. I get right now the message: Unable to open file in editor. vscode-remote://dev-container%2B7b227265706f7369746f727950617468223a227373683a2f2f70616a6540742d73797374656d732d6d6d732e6465407466732e742d73797374656d732d6d6d732e65753a373939392f6167696c652f534f4c494d41542f5f6769742f536f6c696d61742e46726f6e74656e642e416e67756c6172222c22766f6c756d654e616d65223a225f5f554e495155455f5f222c22666f6c646572223a22536f6c696d61742e46726f6e74656e642e416e67756c6172222c22636c6f6e65496e666f223a7b2275726c546f436c6f6e65223a227373683a2f2f70616a6540742d73797374656d732d6d6d732e6465407466732e742d73797374656d732d6d6d732e65753a373939392f6167696c652f534f4c494d41542f5f6769742f536f6c696d61742e46726f6e74656e642e416e67756c6172227d2c22696e7370656374566f6c756d65223a66616c73657d/workspaces/xxx/xxx/xxx/xxx/xxx/xxx/xxx/xxx/xxx/xxx.ts does not map to a local file.
|
From @chris-mcdonald-dev in #1589: Environment (please complete the following information): OS: Windows 11 WSL - Ubuntu Trying to change CSS properties has no effect on the linked CSS file in VS code. Repro steps: Start any server in WSL and open it in the extension's browser. Expected behavior: Expected the updated devtools styles to reflect in VS Code (CSS Mirroring feature). Additional context: This was mentioned in feedback for the initial CSS Mirroring issue (#476), but I wasn't able to find any comments addressing it or references to it in the pinned Known Issues (#1317). Is this something that isn't planned on being fixed? Thanks for everything! |
This issue should be fixed with #1804 so I'll close the issue. Feel free to open a new issue if you're still able to repro the bug. |
If you open a repo using a VS Code remote environment (issue has been tested on WSL + Containers, assuming it's the same in SSH) and try to click on a link to a CSS file in the inspector (source map generated or otherwise) the file fails to load.
After some debugging I found that the following error was being generated:
My debugging shows that the line that the error comes from is https://github.com/microsoft/vscode-edge-devtools/blob/main/src/devtoolsPanel.ts#L309
I'll use comments to detail my findings around a possible fix
The text was updated successfully, but these errors were encountered: