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

feat(ui): open in IDE tests with error on report and code tabs #992

Merged
merged 4 commits into from Mar 21, 2022

Conversation

userquin
Copy link
Member

@userquin userquin commented Mar 20, 2022

Added link and icon only for matching props.file.name, the following capture not showing the editor, I cannot capture 2 windows on Windows, I'm using IntelliJ EE and opening the corresponding file (it seems 2 lines above on the error line):

https://streamable.com/x9ihm5

EDIT: the mouse over also not showing Open in IDE title on span with the icon.

closes #988

@netlify
Copy link

netlify bot commented Mar 20, 2022

✅ Deploy Preview for vitest-dev ready!

🔨 Explore the source changes: 6a91bf8

🔍 Inspect the deploy log: https://app.netlify.com/sites/vitest-dev/deploys/62371e8940b80300084297f5

😎 Browse the preview: https://deploy-preview-992--vitest-dev.netlify.app

@userquin
Copy link
Member Author

userquin commented Mar 20, 2022

maybe we can also include it on the editor tab, testing...

@@ -36,7 +43,16 @@ function relative(p: string) {
{{ task.name }}
<div v-if="task.result?.error" class="scrolls scrolls-rounded task-error">
<pre><b>{{ task.result.error.name || task.result.error.nameStr }}</b>: {{ task.result.error.message }}</pre>
<pre v-for="stack, i of task.result.error.stacks || []" :key="i" op80> - {{ relative(stack.file) }}:{{ stack.line }}:{{ stack.column }}</pre>
<div v-for="(stack, i) of task.result.error.stacks || []" :key="i" class="op80 flex gap-x-2 items-center">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not destructure the stack?

Copy link
Member Author

@userquin userquin Mar 20, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

trying also adding on code tab

@userquin
Copy link
Member Author

working also on code editor

image

@userquin userquin changed the title feat(ui): open in editor error on report tab feat(ui): open in editor error on report and code tabs Mar 20, 2022
@userquin userquin changed the title feat(ui): open in editor error on report and code tabs feat(ui): open in IDE tests with error on report and code tabs Mar 20, 2022
@antfu antfu merged commit c977c33 into main Mar 21, 2022
@antfu antfu deleted the userquin/feat-add-open-editor-on-report branch March 21, 2022 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] Open in editor links
3 participants