Change Details

=== Description Currently there is a max-height on the table of contents (`.sidebar-toc { max-height: 75vh; }`). The purpose of having a max-height is to prevent the table of from extending off the bottom of the screen, (because then it would be impossible to scroll all the way to the bottom of the TOC). The choice of having the max-height be 75vh was to provide make it easier to scroll all the way to the bottom of the TOC when you are at the top of the page (because there is a ~160px of space between the top of the screen and the top of the TOC initially, so if the max-height was 100vh-the sticky offset of the TOC, you wouldn't be able to reach the bottom of the TOC without scrolling the page first, and getting the TOC into it's sticky position). Here is what we have currently, which allows you to scroll to the bottom of the TOC without scrolling the page: {F35546595} However, due to the logic we added in T300973, the table of contents is rarely so tall that you would encounter this situation (this happens more on admin pages like [[ https://en.wikipedia.org/wiki/Wikipedia:Administrators%27_noticeboard/Incidents | wp:ANI ]]). Also, if you scroll the TOC container, and then can't get to the bottom, if you keep trying to scroll without even moving your cursor it will scroll the page. Here is a video of that happening, with the max-height of the TOC set to `calc (98vh - 1.5em)` (`1.5` is the `top`position of the TOC when it is `sticky`): {F35546606} You can see how the scrolling stops before reaching the bottom of the TOC, and then the secondary scroll scrolls the entire page, allowing you to get to the bottom of the TOC. === To do Increase the `max-height` of the TOC to `98vh` minus whatever the top offset is (I think it might be different if you are logged-in, with the sticky header, versus logged-out).

=== Description Currently there is a max-height on the table of contents (`.sidebar-toc { max-height: 75vh; }`). The purpose of having a max-height is to prevent the table of from extending off the bottom of the screen, (because then it would be impossible to scroll all the way to the bottom of the TOC). The choice of having the max-height be 75vh was to make it easier to scroll all the way to the bottom of the TOC when you are at the top of the page (because there is a ~160px of space between the top of the screen and the top of the TOC initially, so if the max-height was 100vh-the sticky offset of the TOC, you wouldn't be able to reach the bottom of the TOC without scrolling the page first, and getting the TOC into it's sticky position). Here is what we have currently, which allows you to scroll to the bottom of the TOC without scrolling the page: {F35546595} However, due to the logic we added in T300973, the table of contents is rarely so tall that you would encounter this situation (this happens more on admin pages like [[ https://en.wikipedia.org/wiki/Wikipedia:Administrators%27_noticeboard/Incidents | wp:ANI ]]). Also, if you scroll the TOC container, and then can't get to the bottom, if you keep trying to scroll without even moving your cursor it will scroll the page. Here is a video of that happening, with the max-height of the TOC set to `calc (98vh - 1.5em)` (`1.5` is the `top`position of the TOC when it is `sticky`): {F35546606} You can see how the scrolling stops before reaching the bottom of the TOC, and then the secondary scroll scrolls the entire page, allowing you to get to the bottom of the TOC. === To do Increase the `max-height` of the TOC to `98vh` minus whatever the top offset is (I think it might be different if you are logged-in, with the sticky header, versus logged-out).

=== Description Currently there is a max-height on the table of contents (`.sidebar-toc { max-height: 75vh; }`). The purpose of having a max-height is to prevent the table of from extending off the bottom of the screen, (because then it would be impossible to scroll all the way to the bottom of the TOC). The choice of having the max-height be 75vh was to provide make it easier to scroll all the way to the bottom of the TOC when you are at the top of the page (because there is a ~160px of space between the top of the screen and the top of the TOC initially, so if the max-height was 100vh-the sticky offset of the TOC, you wouldn't be able to reach the bottom of the TOC without scrolling the page first, and getting the TOC into it's sticky position). Here is what we have currently, which allows you to scroll to the bottom of the TOC without scrolling the page: {F35546595} However, due to the logic we added in T300973, the table of contents is rarely so tall that you would encounter this situation (this happens more on admin pages like [[ https://en.wikipedia.org/wiki/Wikipedia:Administrators%27_noticeboard/Incidents | wp:ANI ]]). Also, if you scroll the TOC container, and then can't get to the bottom, if you keep trying to scroll without even moving your cursor it will scroll the page. Here is a video of that happening, with the max-height of the TOC set to `calc (98vh - 1.5em)` (`1.5` is the `top`position of the TOC when it is `sticky`): {F35546606} You can see how the scrolling stops before reaching the bottom of the TOC, and then the secondary scroll scrolls the entire page, allowing you to get to the bottom of the TOC. === To do Increase the `max-height` of the TOC to `98vh` minus whatever the top offset is (I think it might be different if you are logged-in, with the sticky header, versus logged-out).