? ? Pending

User tests: Successful: Unsuccessful:

avatar anmode
anmode
9 Mar 2022

Pull Request for Issue #37171
Backend Login page -> There's a little space below the left blue bar. I believe this only shows when there are NO buttons in the top header. We have disabled all those buttons for not logged in users

Summary of Changes

Just changed the viewport height so after disabling the login button at the top of the bar.

Testing Instructions

I made changes in the scss folder so after recreating the media folder(creating CSS files) the changes will be reflected on the localhost.

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

3165271449.mp4

Documentation Changes Required

No changes required

Votes

# of Users Experiencing Issue
2/2
Average Importance Score
4.50

avatar anmode anmode - open - 9 Mar 2022
avatar anmode anmode - change - 9 Mar 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Mar 2022
Category Repository NPM Change
avatar anmode anmode - change - 9 Mar 2022
Labels Added: NPM Resource Changed ?
avatar anmode anmode - change - 10 Mar 2022
Title
New commit related to space at bottom of sidebar for non logged user issue #37171
Space at bottom of sidebar for non logged user.
avatar anmode anmode - edited - 10 Mar 2022
avatar mohdabbas955
mohdabbas955 - comment - 10 Mar 2022

I have tested this item successfully on 3f49273

I have tested this item successfully on b1fa223


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

avatar mohdabbas955 mohdabbas955 - test_item - 10 Mar 2022 - Tested successfully
avatar anmode anmode - change - 13 Mar 2022
Title
Space at bottom of sidebar for non logged user.
[4.1] Space at bottom of sidebar for non logged user.
avatar anmode anmode - edited - 13 Mar 2022
avatar Quy
Quy - comment - 13 Mar 2022

Another solution is to add flex-grow-1 to:

<div id="wrapper" class="wrapper">

avatar anmode
anmode - comment - 13 Mar 2022

Another solution is to add flex-grow-1 to:

<div id="wrapper" class="wrapper">

yes, This can be we add here too and that was working fine. what am I supposed to do now? Do I have to add here? or b1fa223 commit is fine

avatar Quy
Quy - comment - 13 Mar 2022

Either use class or css, but not both. Since there is a class for it, use it instead of adding css, but that is just my opinion.

avatar anmode
anmode - comment - 13 Mar 2022

Either use class or CSS, but not both. Since there is a class for it, use it instead of adding CSS, but that is just my opinion.

I just add CSS and the class is already defined may be used somewhere else in the codebase.

avatar Quy
Quy - comment - 13 Mar 2022

flex-grow-1 is a standard Bootstrap class. Either way will work. It is up to the release maintainer to decide.

avatar anmode
anmode - comment - 13 Mar 2022

flex-grow-1 is a standard Bootstrap class. Either way will work. It is up to the release maintainer to decide.

oh okay thanks a lot.

avatar joomla-cms-bot joomla-cms-bot - change - 14 Mar 2022
Category Repository NPM Change Administration Templates (admin)
avatar Quy
Quy - comment - 14 Mar 2022

I have tested this item successfully on 5066719


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

avatar Quy Quy - test_item - 14 Mar 2022 - Tested successfully
avatar Prabhat-10m01
Prabhat-10m01 - comment - 18 Mar 2022

I have tested this item successfully on 5066719

I have tested this item  successfully on 5066719


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

avatar Prabhat-10m01 Prabhat-10m01 - test_item - 18 Mar 2022 - Tested successfully
avatar richard67 richard67 - change - 18 Mar 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 18 Mar 2022

RTC


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

avatar laoneo laoneo - change - 18 Mar 2022
Labels Added: ?
Removed: NPM Resource Changed
avatar laoneo laoneo - change - 18 Mar 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-03-18 12:20:16
Closed_By laoneo
avatar laoneo laoneo - close - 18 Mar 2022
avatar laoneo laoneo - merge - 18 Mar 2022
avatar laoneo
laoneo - comment - 18 Mar 2022

Thanks!!

Add a Comment

Login with GitHub to post a comment