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

Link handler not firing #23149

Closed
dbaeumer opened this issue Mar 24, 2017 · 6 comments
Closed

Link handler not firing #23149

dbaeumer opened this issue Mar 24, 2017 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label verified Verification succeeded
Milestone

Comments

@dbaeumer
Copy link
Member

dbaeumer commented Mar 24, 2017

  • VSCode Version: master

I register the following link handler:

regexp: /^([^\s].*)\((\d+|\d+,\d+|\d+,\d+,\d+,\d+)\):\s+(error|warning|info)\s+(TS\d+)\s*:\s*(.*)$/
matchIndex: 1

no validator.

However no link is computed in the following terminal output:

> tsc -p .
Press any key to close the terminal
HelloWorld.ts(3,17): error TS2339: Property 'loog' does not exist on type 'Console'.

However IMO it should match the line.

@dbaeumer dbaeumer added the terminal General terminal issues that don't fall under another label label Mar 24, 2017
@dbaeumer
Copy link
Member Author

dbaeumer commented Mar 24, 2017

I couldn't even get a simple case like this to work: /(error|warning|info)/

Can you provide me with an example how to use the API to match an error message like

HelloWorld.ts(3,17): error TS2339: Property 'loog' does not exist on type 'Console'.

@dbaeumer dbaeumer added the bug Issue identified by VS Code Team member as probable bug label Mar 24, 2017
@Tyriar
Copy link
Member

Tyriar commented Mar 24, 2017

This worked fine for me:

registerLinkMatcher(/^([^\s].*)\((\d+|\d+,\d+|\d+,\d+,\d+,\d+)\):\s+(error|warning|info)\s+(TS\d+)\s*:\s*(.*)$/, console.log, 1)

Note that you need to hold ctrl/cmd now and also that the hover doesn't yet apply to custom link handlers (this is a bug which I'll fix up now).

@Tyriar Tyriar closed this as completed in ba87991 Mar 24, 2017
@Tyriar
Copy link
Member

Tyriar commented Mar 24, 2017

Should work including the hint now:

image

@Tyriar Tyriar added this to the March 2017 milestone Mar 24, 2017
@dbaeumer
Copy link
Member Author

OK. Sorry for the noise then. I was expecting to see a blue underling link indication when hovering over it.

@Tyriar
Copy link
Member

Tyriar commented Mar 24, 2017

@dbaeumer this changed to align with the editor as much as possible. I didn't want to underline links by default due to lesser line-height than the editor and it could conflict with terminal styles.

@dbaeumer
Copy link
Member Author

I found out why this isn't working for me. See #23262 and #23263

@dbaeumer dbaeumer added the verified Verification succeeded label Mar 31, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants