NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar drmenzelit
drmenzelit
31 Jul 2021

Pull Request for Issue #35003 .

Summary of Changes

Removed debug area for mobile view (I missed that on my last PR)
Added a grid-template-rows definition to make the main content part 1fr in height

Testing Instructions

See issue #35003 and description of PR #34849
Please test with and without modules and content and in different display sizes. Also test with short content and some module in footer position.

Actual result BEFORE applying this Pull Request

See issue

Expected result AFTER applying this Pull Request

The main content is aligned to top no matter if no content is present.
image

Documentation Changes Required

avatar drmenzelit drmenzelit - open - 31 Jul 2021
avatar drmenzelit drmenzelit - change - 31 Jul 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Jul 2021
Category Front End Templates (site) NPM Change
avatar hans2103
hans2103 - comment - 31 Jul 2021

Before

Schermafbeelding 2021-07-31 om 18 23 07

After

Schermafbeelding 2021-07-31 om 18 25 11

avatar hans2103 hans2103 - test_item - 31 Jul 2021 - Tested successfully
avatar hans2103
hans2103 - comment - 31 Jul 2021

I have tested this item successfully on e298184


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

avatar RickR2H
RickR2H - comment - 31 Jul 2021

Problem now is that the bottom positions are also pushed down.

Image-4

The best way is my option 2 ;)

avatar hans2103
hans2103 - comment - 31 Jul 2021

@RickR2H

Problem now is that the bottom positions are also pushed down.
The best way is my option 2 ;)

I don't see this as a problem. The dark blue footer is pushed to the bottom as well.
Feel free to create a PR to fix it Rick.

avatar richard67
richard67 - comment - 31 Jul 2021

@hans2103 Am I blind? I don't see any difference between your before and after screenshots. Ah, no, all ok. Was looking at the wrong ting.

avatar ChristineWk
ChristineWk - comment - 31 Jul 2021

Tried one with demo data. I removed the text from Typography and deactivated the main menu on the right side.
If you want to have an error, you won't get it. :-)

Got this result (without Patch:)

screen shot 2021-07-31 at 21 17 13patch):

Sorry, must stop now. Heavy storm


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

avatar drmenzelit
drmenzelit - comment - 2 Aug 2021

Closing in favour of #35012

avatar drmenzelit drmenzelit - close - 2 Aug 2021
avatar drmenzelit drmenzelit - change - 2 Aug 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-08-02 14:42:11
Closed_By drmenzelit
Labels Added: NPM Resource Changed ?

Add a Comment

Login with GitHub to post a comment