kestra icon indicating copy to clipboard operation
kestra copied to clipboard

Plural errors in the flow are chained without any separator making it difficult to decipher.

Open ReLoutre opened this issue 10 months ago • 3 comments

Image

ReLoutre avatar Jun 10 '25 08:06 ReLoutre

@ReLoutre , how's this ?

Image

satvik2131 avatar Jun 17 '25 08:06 satvik2131

Hi @satvik2131 ! Already better indeed, could be good to test the display with errors (as long as possible, you may increase the id length if the task I.E, to test the wrapping) and in the same time with warnings.

ReLoutre avatar Jun 17 '25 08:06 ReLoutre

Also, could be more readable for long errors to separate them, something like that :

Image

Top notch addition could be to have a scroll bar i guess, but let's wait before trying to implement it.

ReLoutre avatar Jun 17 '25 10:06 ReLoutre

@ReLoutre , if we implement this separation then individual scrollbar in every error would appear ,would that be fine , also the error string comes as one , so need to do some string manupulation , I have enable scrollbar in the whole error box . Let me know which approach to go with ?

https://github.com/user-attachments/assets/40509712-3199-46de-a123-5f08506ffdfb

satvik2131 avatar Jun 19 '25 04:06 satvik2131

When referring to scroll bar i was thinking of a vertical one, to navigate through all the errors. What's more, i think we should use wrapping of words to avoid having super long lines. Image

ReLoutre avatar Jun 19 '25 09:06 ReLoutre

I can work on the vertical scrolling part and wrapping as well but for wrapping of words I need some guidance how we wanted to process it : like creating a regex to text function or using custom messages instead of these regex patterns . Please mention the approach to go further and will start working on it .

satvik2131 avatar Jun 19 '25 18:06 satvik2131

Would be best to not reinvent the wheel here, and use the current word wrapping technique already used in the application (like Logs, Search, YAML Editor, No Code Editor, etc.)

Ben8t avatar Jun 20 '25 06:06 Ben8t