Thank you for working on the ticket @rodbv
I like the way the solution looks!
The original ticket is concerned about the memory usage of such a big page (#10919 (Add an option to disable display of related items on admin's delete confirmation page (to prevent large memory usage on complex objects)) – Django), therefore adding a scrollbar does not resolve the reported issue imo.
I am curious what is the accessibility concern is for having the “Yes, I’m sure” at the bottom of a large page that would require scrolling? If the accessibility team feels this makes the page unusable for folks using assistive technologies, then I would even be in favor of us implementing a delete_confirmation_max_display value by default.
I would vote for a global limit.
I agree that folks should feel empowered to overwrite pages as they see fit. For example, if I was customizing it, I would have a load more/paginated approach using a bit of htmx, but for Django core, that’s overkill.
I feel this problem has been felt by folks every now and again and setting 1 attribute to resolve this is probably welcome.