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

Test VS Live Server/Preview - opening a file using a link from the terminal #62

Closed
lszomoru opened this issue Jun 29, 2021 · 5 comments · Fixed by #38
Closed

Test VS Live Server/Preview - opening a file using a link from the terminal #62

lszomoru opened this issue Jun 29, 2021 · 5 comments · Fixed by #38
Assignees
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@lszomoru
Copy link
Member

Testing microsoft/vscode#127164

When clicking on the link to a file in the terminal, I am wondering whether we should be opening this file in a separate editor group than the preview itself. This way, you can see both the file that you have chosen to open as well as the preview itself. We could also make this configurable to give users more flexibility on how the files are being opened.

@rzhao271
Copy link

rzhao271 commented Jul 2, 2021

@andreamah what are some steps to verify this feature?

@andreamah
Copy link
Contributor

I just pushed some fixes to the terminal link provider to the newest version a few minutes ago, so please upgrade to v0.1.7!

Steps:

  1. Run the task Live Preview: Run Server --verbose.
  2. Open the internal preview of an html page and interact with some of the links to navigate to other pages and load content. The task terminal should be logging server traffic.
  3. Ensure that the active editor is the preview webview.
  4. Try clicking on the filenames in the terminal as you would with a hyperlink (ctrl+click)
  5. The file should either open up in an existing editor group beside the webview or open a new editor group to the right in a single-column scenario. It should not open in the same editor group as the embedded preview.

@rzhao271
Copy link

rzhao271 commented Jul 2, 2021

I got the tab to show up in the same editor group in the following case:

  1. Have two editor groups, one with a file, and one with the preview.
  2. Close the file. This closes that editor group, and the preview remains.
  3. Click a link in the terminal.

Screencap demonstrating steps above

@rzhao271 rzhao271 reopened this Jul 2, 2021
@andreamah
Copy link
Contributor

Good catch! I checked and seems that the panel's viewcolumn was not updated by the time that the open was performed in this specific case... I'll need to look into this.

@andreamah andreamah transferred this issue from microsoft/vscode Jul 9, 2021
@andreamah andreamah added feature-request Request for new features or functionality verification-found Issue verification failed labels Jul 9, 2021
@andreamah andreamah added this to the Backlog milestone Dec 7, 2022
@andreamah andreamah self-assigned this Dec 7, 2022
@andreamah
Copy link
Contributor

fixed upstream with microsoft/vscode#131677

@rzhao271 rzhao271 modified the milestones: Backlog, January 2023 Jan 26, 2023
@rzhao271 rzhao271 added verification-needed Verification of issue is requested verified Verification succeeded and removed verification-found Issue verification failed labels Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants