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

extra characters are added to the links in the terminal #36806

Closed
capaj opened this issue Oct 24, 2017 · 1 comment
Closed

extra characters are added to the links in the terminal #36806

capaj opened this issue Oct 24, 2017 · 1 comment
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code)

Comments

@capaj
Copy link

capaj commented Oct 24, 2017

  • VSCode Version: Version 1.17.2
    Commit b813d12
    Date 2017-10-16T13:57:00.652Z
    Shell 1.7.7
    Renderer 58.0.3029.110
    Node 7.9.0
    Architecture x64
  • OS Version: ubuntu 17.10

Steps to Reproduce:

  1. print an output like this into the terminal:
templateData:  { event:
   { title: 'event one',
     location: 'room 1',
     description: 'not much 1',
     when: 'Mon. Nov 20, 11pm - 11 pm',
     timezone: 'BST',
     link:
      { yes: 'http://looopco.loc.dev:3000/#/event/1?confirm',
        no: 'http://looopco.loc.dev:3000/#/event/1?confirm=false' } } }
  1. click on one of those URLs while holding down ctrl

expected result:
a proper link is opened

actual result:
a link is opened, but it has an extra ', for the yes link and ' for the no link

Reproduces without extensions: Yes

probably the regex used for the links needs tweaking

@vscodebot vscodebot bot added the terminal General terminal issues that don't fall under another label label Oct 24, 2017
@Tyriar
Copy link
Member

Tyriar commented Oct 24, 2017

Unfortunately the ' character is valid in URIs which is why this is happening. To support this we'll need to use a proper parser for link detection, not the simple regex-based system currently being used. Closing this in favor of xtermjs/xterm.js#583, subscribe there for updates.

@Tyriar Tyriar closed this as completed Oct 24, 2017
@Tyriar Tyriar added *duplicate Issue identified as a duplicate of another issue(s) upstream Issue identified as 'upstream' component related (exists outside of VS Code) labels Oct 24, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

3 participants