annotation-tool Fix height of container for creating new annotations

This PR should fix https://github.com/opencast/annotation-tool/issues/630 .

For some reason the default height for the container containing the labels was set to 100vh. I set this to auto so that it won't use the whole screen height.

I also changed the value for overflow-y to auto so that the scrollbar for labels is only visible, when neccessary.

Dec 02 '24 11:12 jduehring

It now works as expected. Thx for the fix!

Dec 03 '24 20:12 dagraf

@JulianKniephoff The requirement as I see it would be: "A user can see the options to stop the video when this user does freshly load a video." I can live with the solution as it is right now with but I am open to improvements. Do you have here an improvement in mind that could get implemented easily?

Dec 05 '24 13:12 dagraf

I don't really have anything specific in mind. But yeah, the requirement as you stated it is not met by this change, when you have a sufficient amount of labels in a category, right? Am I doing something wrong? 🤔

Dec 11 '24 15:12 JulianKniephoff

Right. But as you do not have a specific improvement in mind which can meet the stated requirement and as I said that I can live with the current solution: We implement this solution and close this ticket.

Dec 11 '24 16:12 dagraf

Fair. :)

I would still say that we should just get rid of .categories-container-scroll in that case. Except @jduehring tells me it does actually change things.

Dec 12 '24 15:12 JulianKniephoff

@JulianKniephoff Agreed, I removed the container and as you said it doesn't seem to change anything else.

Dec 17 '24 10:12 jduehring