Changes between Initial Version and Version 1 of Ticket #31576, comment 3
- Timestamp:
- May 13, 2020, 4:19:06 AM (5 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #31576, comment 3
initial v1 1 1 I'm not sure that we need go as far as removing the sidebar for mid-sized screens. Shutting the sidebar is sufficient to ''resolve'' the issue, and it's a nice improvement. 2 2 3 Similar discussion on the PR: [https://github.com/django/django/pull/12159#issuecomment-626703045 Tom], [https://github.com/django/django/pull/12159#issuecomment-626711533 Mariusz]. That was to do with inline scrolling but with the same conclusion, "... t can be mitigated by just closing it, so I think it's alright from the user's perspective".3 Similar discussion on the PR: [https://github.com/django/django/pull/12159#issuecomment-626703045 Tom], [https://github.com/django/django/pull/12159#issuecomment-626711533 Mariusz]. That was to do with inline scrolling but with the same conclusion, "...it can be mitigated by just closing it, so I think it's alright from the user's perspective". 4 4 5 5 The error here only shows up on the selenium headless tests. Actually using the admin on a 800px viewport is fine, great even: you just close the sidebar if you don't want it.