Same as FIX: Software update prompt position on themes with extra header by pmusaraj · Pull Request #15028 · discourse/discourse · GitHub, but here I am skipping the failing test instead of trying to fix it.
@Flink I think we should skip this test for now because it doesn’t actually test “sticky avatars when scrolling up”, rather it tests sticky avatars for tall posts. I confirmed this on
main by removing the second scrollTo line in the test. It will still pass, because that specific post satisfied the “postContentHeight > windowheight - header height” condition. Also, because in tests the app does not take up the full window height, but is instead in a resized and zoomed-out container, the test has to scroll the container and not the window, which is not the same behaviour as the app. (I tried to fix this in the original PR, but ran into one stubborn issue with Firefox 78.)