• f020cbb9 Merge pull request #14 from n-peugnet/fix-scroll-regression […]

In reply to @n:matrix.nbios.net
Hi everyone, I'm using gomuks 3.0-2, I have a crash on /cs fetch after inserting the key, am I the only one?

Adding a break seems a good idea, as we don't need to check the rest of the children if we had one match. But another problem I see with this code is that &child does not reference the value from flex.children, but rather its copied value from the range loop. It should probably look something like this instead:

-	for _, child := range flex.children {
-		if child.target == comp {
-			flex.focused = &child
+	for i := range flex.children {
+		childp = &flex.children[i]
+		if childp.target == comp {
+			flex.focused = childp
			flex.focused.Focus()
			break
		}
	}

In reply to @fractivore:cyberia.club
Did gomuks politely give you the location of a file that logged the stack trace when it crashed? If so, that may help. Otherwise, /tmp/gomuks/debug.log might help.

Is It standard for terminals to support links in the proper way?

In Kitty, the style of text is separate from whether the text is clickable. Hope that makes sense.

There is no "the proper way" methinks.