woodpecker-ci / woodpecker

Woodpecker is a simple, yet powerful CI/CD engine with great extensibility.
https://woodpecker-ci.org
Apache License 2.0
4.1k stars 354 forks source link

Terminal issues #1012

Closed anbraten closed 2 years ago

anbraten commented 2 years ago

Component

web-ui

Describe the bug

I noticed some issues with the new terminal:

Screenshot from 2022-07-05 09-28-45

Screenshot from 2022-07-05 09-27-57

System Info

{"source":"https://github.com/woodpecker-ci/woodpecker","version":"next-3f73d5bf"}

Additional context

No response

Validations

qwerty287 commented 2 years ago

I also noticed on mobile, if I click into the terminal area, my keyboard appears, but I can't type anything (obviously).

thestr4ng3r commented 2 years ago

log lines for other running jobs shown in selected running jobs? 🤔 or the terminal is not properly reset after switching

I think this always happens while the job is still running, when it is streaming the log in realtime rather than just fetching it statically.

For our instance, we have switched to using xterm's dom renderer instead of the canvas-based one since the canvas one feels unresponsive to me when scrolling (always a bit delayed): https://github.com/rizinorg/woodpecker/commit/6146b70e5f858643532078052570aeae6ca0cb51#diff-a381d47772869b92cbbfb7b65a6e4222af8629b6a0b5fd5ba7db7e12dd5bff4aR102 This generates a lot of dom, but might be worth trying out too.

qwerty287 commented 2 years ago
  1. The green Exit-Code 0 is not good visible in light mode.
  2. Also https://github.com/woodpecker-ci/woodpecker/issues/998