swagger-editor icon indicating copy to clipboard operation
swagger-editor copied to clipboard

Make window with errors expandable

Open savinmikhail opened this issue 9 months ago • 6 comments

Can we make bottom window with errors and warnings expandable, just like right tab with rendered doc? Now it is small, and not handy to investigate or overview errors :(

Image

savinmikhail avatar Apr 17 '25 08:04 savinmikhail

Hello there @savinmikhail. Thank you for creating an issue. Are you open to providing a PR?

robert-hebel-sb avatar Apr 17 '25 11:04 robert-hebel-sb

@robert-hebel-sb hi, unfortunately no, I have no experience in frontend :(

savinmikhail avatar Apr 17 '25 11:04 savinmikhail

Hi @robert-hebel-sb, I would like to work on solving this issue.

Miran-Firdausi avatar Oct 24 '25 07:10 Miran-Firdausi

hello there @Miran-Firdausi 👋

sure, I've assigned the issue to you, and I'm looking forward to PR 💟

robert-hebel-sb avatar Oct 30 '25 17:10 robert-hebel-sb

Image

Hi @robert-hebel-sb,

I used the same logic that was applied to create the left and right panes, but I found one issue. With react-split-pane, it’s not possible to collapse a pane completely, there’s always a small gap left, even when the pane has no content.

I tried conditionally rendering the SplitPane and just the Editor, but that didn’t work either, as it caused the underlying text model to be destroyed on re-render.

So, one option is to always keep the error pane visible, even when it’s empty. Or would you prefer that I look for another package to handle this?

Also, there are ways to manipulate DOM to get it done, but that wont be very neat. What's your advice?

Miran-Firdausi avatar Nov 01 '25 20:11 Miran-Firdausi

hey @Miran-Firdausi 👋

From the issue attachment, I assume the fix should be for swagger-editor@next, which uses the react-collapse-pane library.

I haven't checked it, but according to docs, they support collapsed mode https://b-zurg.github.io/react-collapse-pane/#/?id=react-collapse-pane

robert-hebel-sb avatar Nov 03 '25 14:11 robert-hebel-sb