search

opening-up-chatgpt / opening-up-chatgpt.github.io

Tracking instruction-tuned LLM openness. Paper: Liesenfeld, Andreas, Alianda Lopez, and Mark Dingemanse. 2023. “Opening up ChatGPT: Tracking Openness, Transparency, and Accountability in Instruction-Tuned Text Generators.” In Proceedings of the 5th International Conference on Conversational User Interfaces. doi:10.1145/3571884.3604316.
https://opening-up-chatgpt.github.io/
Apache License 2.0
80 stars 5 forks source link

Update sticky table header style in styles.css #96

Closed asabuncuoglu13 closed 1 week ago

asabuncuoglu13 commented 1 week ago

Current behaviour of the web page:

The header moves all the way up with the content, which makes it hard to view features for the models.

Suggested behaviour:

Sticky header position fixed at the top when moved.

mdingemanse commented 1 week ago

Hey, thanks — works for me on desktop in Firefox. Not on mobile AFAICS and my HTML/CSS is too rusty to figure out how/why. Do you have capacity to test this? Not a breaking issue as your contribution is a great improvement on desktop anyway!

asabuncuoglu13 commented 1 week ago

Hey, no worries! I also checked it now and encountered the same issue. The problem is the overflow-x behaviour of div#included-table (when the width is less than 1200px) breaks the sticky header position. I think you can safely remove the overflow-x: auto line. Because you are already resizing the entire table for responsive behaviour, so you expect to see the same overflow behaviour with the desktop view.