Expand only PR merge status list to fit all items #4124
jameschensmith posted onGitHub
Summary
There was previously introduced a CSS rule to show all PR merge status items instead of scrollable div. The original intention was using it for PR merge statuses, but this CSS rule also applies to the modals as well (in some cases, it doesn't due to the class open
not being applied). For repositories with a sizable merge status list (npm/arborist is the example in the screenshot), the height of the list without a scrollbar is quite silly :yum:
I would like to recommend a more modest size (maybe up to 10 items?). I understand the original reason to make it longer, but with no overflow the user will still have to scroll (the page), which defeats the purpose of the original intention of showing all items :sweat_smile:
I would also like to expand on this (in the spirit of this repository), and suggest a future potential enhancement. Maybe this would turn into its own feature. It may be nice to add the ability to resize the merge status list.
This is my first issue in this repository, by the way. Thank you for developing this! Why did I not start using this sooner! :blush: :clap:
Screenshot(s)
<details> <summary>Long merge status list</summary>
</details>
References
Relates to #1004.