? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
28 Aug 2021

Pull Request for Issue #35262

Summary of Changes

Kudos to @brianteeman for identifying the root cause and the fix for this - Thanks.

Correctly calculate the minimum height of the sidebar to avoid any blank space at the bottom of the page

Testing Instructions

see #35262 - if you can replicate that then try it before and after applying this PR

Actual result BEFORE applying this Pull Request

A slither of space in at the very bottom when the window size is very large

130267151-2c615645-bc02-454b-845c-b307648eb5ca

Expected result AFTER applying this Pull Request

sidebar reaches the very bottom

Documentation Changes Required

avatar PhilETaylor PhilETaylor - open - 28 Aug 2021
avatar PhilETaylor PhilETaylor - change - 28 Aug 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 28 Aug 2021
avatar joomla-cms-bot joomla-cms-bot - change - 28 Aug 2021
Category Administration Templates (admin) NPM Change
avatar RickR2H RickR2H - test_item - 29 Aug 2021 - Tested successfully
avatar RickR2H
RickR2H - comment - 29 Aug 2021

I have tested this item successfully on 324a8ec

Did notice the behavior before. I was only a minor annoyance, but glad it's fixed!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35412.

avatar Quy Quy - alter_testresult - 31 Aug 2021 - Chaosxmk: Tested successfully
avatar Quy Quy - change - 31 Aug 2021
Status New Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 31 Aug 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35412.

avatar Quy Quy - close - 31 Aug 2021
avatar Quy Quy - merge - 31 Aug 2021
avatar Quy Quy - change - 31 Aug 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-08-31 15:24:16
Closed_By Quy
Labels Added: ?
avatar Quy
Quy - comment - 31 Aug 2021

Thank you all!

Add a Comment

Login with GitHub to post a comment