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

Output panel not support for virtual scrolling #7584

Open
ybom77 opened this issue Nov 2, 2023 · 4 comments
Open

Output panel not support for virtual scrolling #7584

ybom77 opened this issue Nov 2, 2023 · 4 comments
Labels
in linear Issue or PR has been created in Linear for internal review

Comments

@ybom77
Copy link

ybom77 commented Nov 2, 2023

Describe the bug

When I request very large JSON data using request node, when I look at the output panel, I see that virtual scrolling is not supported, but the library "vue-json-pretty" used is supporting virtual scrolling settings. So I think this should be a BUG

Expected behavior

Should support virtual scrolling to view data
Environment (please complete the following information):

  • n8n Version [e.g. 1.14.0]
@Joffcom
Copy link
Member

Joffcom commented Nov 2, 2023

Hey @ybom77,

Can you be a bit clearer? I am not sure what the bug is that you are reporting or how I would try to reproduce it. I would say though becuase a package or library supports something and we have chosen to not implement it I don't think that makes it a bug, It all depends on if something is broken or not.

@Joffcom Joffcom self-assigned this Nov 2, 2023
@ybom77
Copy link
Author

ybom77 commented Nov 2, 2023

hey @Joffcom

image

In the output panel, when my data is very large, it will cause the Browser interface stuttering.

Should we consider enabling virtual scrolling ,the "vue-json-pretty" lib already supports it.

@Joffcom
Copy link
Member

Joffcom commented Nov 2, 2023

Hey @ybom77,

Thanks for that, so would say as nothing is broken this is not a bug and is instead an enhancement to improve the experience.

I will get this added to Linear shortly to see if this is something we want to implement soon.

@Joffcom
Copy link
Member

Joffcom commented Nov 3, 2023

I have created N8N-7034 as the internal ticket to think about this one.

@Joffcom Joffcom removed their assignment Nov 3, 2023
@Joffcom Joffcom added the in linear Issue or PR has been created in Linear for internal review label Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in linear Issue or PR has been created in Linear for internal review
Projects
None yet
Development

No branches or pull requests

2 participants