?
avatar PhilETaylor
PhilETaylor
29 Dec 2017

Steps to reproduce the issue

4.0-dev install
login frontend as super admin
click Template Settings
Click Select on the logo media selector

Expected result

Full width grey background under modal

Actual result

grey background clipped at edges

(Note Im on HALF a 27" screen)

screen shot 2017-12-29 at 18 14 17

System information (as much as possible)

Additional comments

avatar PhilETaylor PhilETaylor - open - 29 Dec 2017
avatar joomla-cms-bot joomla-cms-bot - change - 29 Dec 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 29 Dec 2017
avatar PhilETaylor PhilETaylor - change - 29 Dec 2017
The description was changed
avatar PhilETaylor PhilETaylor - edited - 29 Dec 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 29 Dec 2017
Build staging 4.0-dev
avatar franz-wohlkoenig franz-wohlkoenig - change - 29 Dec 2017
Category Administration Front End Templates (site)
avatar franz-wohlkoenig franz-wohlkoenig - change - 29 Dec 2017
Status New Confirmed
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 29 Dec 2017

Issue confirmed:

bildschirmfoto 2017-12-29 um 19 30 31

System information

4.0.0-dev
Sample Data
macOS Sierra, 10.12.6
Firefox 57 (64-bit)

MAMP 4.2

  • Apache-Server
  • PHP 7.0.22
  • MySQLi 5.6.35
avatar C-Lodder
C-Lodder - comment - 30 Dec 2017

@ciar4n this is causing the issue: https://github.com/joomla/joomla-cms/blob/4.0-dev/templates/cassiopeia/scss/blocks/_layout.scss#L11

As far as I can tell, all child element of the <body> use the .container-* class, so seeing as you added:

https://github.com/joomla/joomla-cms/blob/4.0-dev/templates/cassiopeia/scss/blocks/_css-grid.scss#L22

can we remove max-width: 1080px + ($cassiopeia-grid-gutter * 4);?

avatar Anu1601CS
Anu1601CS - comment - 30 Dec 2017

@C-Lodder if we remove max-width: 1080px + ($cassiopeia-grid-gutter * 4); is this issue solve ??


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

avatar C-Lodder
C-Lodder - comment - 30 Dec 2017

@Anu1601CS - yes this does resolve the issue, however Ciaran wrote the css-grid, so he may have added this for a reason. Don't want to remove it until I get confirmation from him. If it needs to stay, then we can replace this:

.site {
  background-color: #fafafa;

  > div {... }
}

with:

.site {
  background-color: #fafafa;

  > div:not(.modal-backdrop) {... }
}

although I'm not a fan of that approach

avatar Anu1601CS
Anu1601CS - comment - 30 Dec 2017

@C-Lodder ok thanks
What about at this line


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

avatar C-Lodder
C-Lodder - comment - 30 Dec 2017

That's what the SCSS compiles to. We don't edit CSS files, we edit the SCSS files then compile it

avatar Anu1601CS
Anu1601CS - comment - 30 Dec 2017

ok thanks


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

avatar C-Lodder
C-Lodder - comment - 31 Dec 2017

PR: #19241

avatar joomla-cms-bot joomla-cms-bot - change - 31 Dec 2017
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 31 Dec 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 31 Dec 2017
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2017-12-31 12:40:22
Closed_By franz-wohlkoenig
avatar joomla-cms-bot
joomla-cms-bot - comment - 31 Dec 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Dec 2017

closed as having Pull Request #19241

Add a Comment

Login with GitHub to post a comment