Feature/tooltip autoscroll

[tonihele]

Little bit quirky. Implements the tooltip autoscroll if it doesn't entirely fit to the allocated space.

• Somehow I could not get the style applied properly, so the component now has a fixed style set. Not a problem for us but annoying design
• The effect length is 70s, hmm, yeah, the effects time calculation is a bit difficult for this kind of task. I would have wanted infinite time but then we just get 1s always as normalized time. So now everything is normalized to this 70s, the delays and speed. Pretty annoying. The effect length should perhaps be take account in the math so that this length variable could be adjusted freely without the effect speed changing....

Edit: I fixed the math so that there are far less variables and more constants

[Copilot]

Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 suggestion.

Files not reviewed (2)

• assets/Interface/GameHUD.xml: Language not supported
• assets/Interface/MyControls.xml: Language not supported

Comments skipped due to low confidence (2)

src/toniarts/openkeeper/gui/nifty/autoscrolltext/AutoScrollTextControl.java:43

• The logger should be named 'LOGGER' to follow Java naming conventions for constants.

private static final Logger logger = System.getLogger(AutoScrollTextControl.class.getName());

src/toniarts/openkeeper/view/SystemMessageState.java:128

• The indentation of the closing brace is incorrect. It should be aligned with the opening brace of the method.

    }

[Copilot]

The speedFactor property should be parsed as a float instead of an integer for more precise control over the speed.

Suggested change

	speedFactor = Integer.parseInt(parameter.getProperty("length", "1000"));
	speedFactor = Float.parseFloat(parameter.getProperty("length", "1000"));