Changes between Initial Version and Version 1 of Ticket #31576, comment 3


Ignore:
Timestamp:
May 13, 2020, 4:19:06 AM (5 years ago)
Author:
Carlton Gibson

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #31576, comment 3

    initial v1  
    11I'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.
    22
    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".
     3Similar 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".
    44
    55The 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.
Back to Top