leomoty

I am not really able to reproduce. The selection with cursor is broken when you rotate the terminal.

Related: https://github.com/cockpit-project/cockpit/pull/13250

@ApertureDevelopment I am not actually a cockpit developer, just got you some context. Glad that it worked out for you :)

@git-Hmmm this seems available now: https://github.com/cockpit-project/cockpit/pull/16958 Part of cockpit since 264

Sure, will take a look this weekend. Any ideas why it works on firefox but not chrome?

So, I was able to narrow down the issue, it only fails on split view: https://github.com/cockpit-project/cockpit/pull/9943/files?utf8=%E2%9C%93&diff=split#diff-a2edaced83836966f97b3d8df14beabcR24 It happens in both chrome and firefox.

@stefanbuck I had a hunch. When in split view, the HTML structure is as follow: ``` var utils = require("./utils.js"); } from "./dialogx.jsx"; ``` The regex is not really effective,...

Tried running this by hand: https://github.com/padolsey/findAndReplaceDOMText/blob/master/src/findAndReplaceDOMText.js#L262 Unless I am mistaken, the result is what I expected: ``` ... import cockpit from "cockpit"; import { var cockpit = require("cockpit"); dialog_open, TextInput,...

Ya me neither, unless we somehow start treating each side in split diff view as a different blob. Right now, that approach would break and it requires more than just...

It seems that the multiline import does no longer exist in that PR. I'll try to find another one. Edit: https://github.com/cockpit-project/cockpit/commit/24ccf2e5e51ce64dc661d341f25a64a806294f4a?diff=split#diff-fcaf1080ed0a2139de7cef9b184476a6L71