addjust footer height when legal links are present. fixes #9572
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
This commit is contained in:
parent
323b665e88
commit
ac22c28045
|
@ -155,3 +155,9 @@ input.primary,
|
|||
border: 1px solid #ebebeb;
|
||||
}
|
||||
}
|
||||
|
||||
@if ($has-legal-links == 'true') {
|
||||
footer {
|
||||
height: 92px;
|
||||
}
|
||||
}
|
||||
|
|
|
@ -143,11 +143,11 @@ class ThemingDefaults extends \OC_Defaults {
|
|||
}
|
||||
|
||||
public function getImprintUrl() {
|
||||
return $this->config->getAppValue('theming', 'imprintUrl', '');
|
||||
return (string)$this->config->getAppValue('theming', 'imprintUrl', '');
|
||||
}
|
||||
|
||||
public function getPrivacyUrl() {
|
||||
return $this->config->getAppValue('theming', 'privacyUrl', '');
|
||||
return (string)$this->config->getAppValue('theming', 'privacyUrl', '');
|
||||
}
|
||||
|
||||
public function getShortFooter() {
|
||||
|
@ -290,6 +290,12 @@ class ThemingDefaults extends \OC_Defaults {
|
|||
if ($this->config->getAppValue('theming', 'backgroundMime', null) === 'backgroundColor') {
|
||||
$variables['image-login-plain'] = 'true';
|
||||
}
|
||||
|
||||
$variables['has-legal-links'] = 'false';
|
||||
if($this->getImprintUrl() !== '' || $this->getPrivacyUrl() !== '') {
|
||||
$variables['has-legal-links'] = 'true';
|
||||
}
|
||||
|
||||
$cache->set('getScssVariables', $variables);
|
||||
return $variables;
|
||||
}
|
||||
|
|
|
@ -670,7 +670,8 @@ class ThemingDefaultsTest extends TestCase {
|
|||
'color-primary' => $this->defaults->getColorPrimary(),
|
||||
'color-primary-text' => '#ffffff',
|
||||
'image-login-plain' => 'false',
|
||||
'color-primary-element' => '#aaaaaa'
|
||||
'color-primary-element' => '#aaaaaa',
|
||||
'has-legal-links' => 'false'
|
||||
|
||||
];
|
||||
$this->assertEquals($expected, $this->template->getScssVariables());
|
||||
|
|
Loading…
Reference in New Issue