? Pending

User tests: Successful: Unsuccessful:

avatar astridx
astridx
8 Sep 2019

Pull Request for Issue #26201.

Summary of Changes

No margin for sidebar left.

Testing Instructions

  1. Apply this patch and run node build.js --compile-css for updating the changed scss file. Alternatively you can run npm i.
  2. Add a module to the left sidebar.
  3. Open the side in the front end and see, that the Cassiopeia sidebars and component area horizontally aligned.

Expected result

Cassiopeia sidebars and component area horizontally aligned.

Actual result

Cassiopeia sidebars and component area not horizontally aligned.

Documentation Changes Required

No

avatar astridx astridx - open - 8 Sep 2019
avatar astridx astridx - change - 8 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Sep 2019
Category Front End Templates (site)
avatar Scrabble96
Scrabble96 - comment - 8 Sep 2019

This PR needs to include no top margin for the container-component as well. As I said in my issue #26201, if there is no left sidebar but there is a right sidebar, then the margin is applied to the component area and the right sidebar appears higher than the component (at least it does on larger screens; on the smallest screens both left and right sidebars drop down below the component area.)

avatar astridx astridx - change - 8 Sep 2019
Labels Added: ?
avatar astridx astridx - change - 8 Sep 2019
The description was changed
avatar astridx astridx - edited - 8 Sep 2019
avatar astridx
astridx - comment - 8 Sep 2019

@Scrabble96 You are right. I added the missing class now.

avatar ChristineWk
ChristineWk - comment - 8 Sep 2019

@astridx As I don't know about npm /node, I tried as follows:
Changed in template.css (line 12349) from the file: a236fc6 accordingly.

But I think there is missing after: margin-bottom:20px; } Because the: + div becomes:red instead green. So, I added the missing }
Unfortunately still Actual Result. Also I don't know how to compile it to scss file. Sorry.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26210.
avatar franz-wohlkoenig franz-wohlkoenig - change - 9 Sep 2019
Title
left sidebar align
[4.0] left sidebar align
avatar franz-wohlkoenig franz-wohlkoenig - edited - 9 Sep 2019
avatar astridx
astridx - comment - 9 Sep 2019

@ChristineWk Thank you for trying to test. If you like to make you familiar with the environment this is a good entry for windows users: https://www.wicked-software.de/joomla-v4-0-win10-composer-and-npm

avatar ChristineWk
ChristineWk - comment - 9 Sep 2019

@astridx Thank you for your info - I hv to study :-)

just OT: On my test-site (using user.css) entered:

.site-grid .container-header + div:not(.container-banner) {
margin-top: 0px;
}
it looks OK.
regards, christine

62815fa 1 Oct 2019 avatar astridx wip
5a22d16 1 Oct 2019 avatar astridx wip
avatar ChristineWk
ChristineWk - comment - 1 Oct 2019

Hi Astrid,

Thanks for adding commits. After activating under Options CI Server Settings > Switch CI Integration > ON unfortenately (still):
The patch could not be applied either due to missing connection to the server or missing patch on the server.

rgds Christine


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

avatar ChristineWk
ChristineWk - comment - 1 Oct 2019

Hi Astrid,

Thanks for adding commits. After activating under Options CI Server Settings > Switch CI Integration > ON unfortenately (still):
The patch could not be applied either due to missing connection to the server or missing patch on the server.

rgds Christine


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

avatar ChristineWk
ChristineWk - comment - 2 Oct 2019

The reason for might be:
"This branch is out-of-date with the base branch ..."

avatar brianteeman
brianteeman - comment - 2 Oct 2019

No that would not be the issue

avatar ChristineWk
ChristineWk - comment - 2 Oct 2019

@brianteeman

Thank you for info. What is the issue for? Maybe: "Some checks were not successful. Build is failing ...."
If you think there is no way, then I would have to stop the activities in this regard.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26210.
avatar Kaustubharas Kaustubharas - test_item - 19 Oct 2019 - Tested unsuccessfully
avatar Kaustubharas
Kaustubharas - comment - 19 Oct 2019

I have tested this item ? unsuccessfully on 068cc4a

Cassiopeia sidebars and component area not horizontally aligned.

When a module is present at sidebar-left position, the margin-top: 15px applies to it.

And, when there is no sidebar-left position, the margin-top: 15px applies to the component area,


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

avatar patjun
patjun - comment - 19 Oct 2019

without patch
without-patch

patch applied
patch-applied

avatar patjun patjun - test_item - 19 Oct 2019 - Tested successfully
avatar patjun
patjun - comment - 19 Oct 2019

I have tested this item successfully on 068cc4a


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

avatar sanderpotjer sanderpotjer - test_item - 19 Oct 2019 - Tested successfully
avatar sanderpotjer
sanderpotjer - comment - 19 Oct 2019

I have tested this item successfully on 068cc4a

After test sidebar is nicely aligned. Thanks @astridx


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

avatar wilsonge wilsonge - change - 19 Oct 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-19 22:53:29
Closed_By wilsonge
avatar wilsonge wilsonge - close - 19 Oct 2019
avatar wilsonge wilsonge - merge - 19 Oct 2019
avatar wilsonge
wilsonge - comment - 19 Oct 2019

Thanks!

avatar Scrabble96
Scrabble96 - comment - 20 Oct 2019

Thank you :-)

Add a Comment

Login with GitHub to post a comment