This is an automated email from the ASF dual-hosted git repository. michaelo pushed a commit to branch fix-padding in repository https://gitbox.apache.org/repos/asf/maven-fluido-skin.git
commit ad8e01be560b9367cba156c9f760901581c78b90 Author: Michael Osipov <[email protected]> AuthorDate: Sun Jul 2 23:33:01 2023 +0200 [MSKINS-234] Disabling 'topBarEnabled' doesn't remove padding This closes #53 --- src/main/resources/META-INF/maven/site.vm | 6 +++--- src/main/resources/css/maven-theme.css | 10 +++++++--- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/src/main/resources/META-INF/maven/site.vm b/src/main/resources/META-INF/maven/site.vm index fa1a71a..4e5f1d2 100644 --- a/src/main/resources/META-INF/maven/site.vm +++ b/src/main/resources/META-INF/maven/site.vm @@ -60,7 +60,7 @@ #**##if ( $topBarEnabled ) <body class="topBarEnabled"> #**##else - <body class="topBarDisabled"> + <body> #**##end #**##forkMeOnGitHub() #**##facebookLoadSDK() @@ -177,9 +177,9 @@ #**##end ## #**##if ( $sideBarEnabled ) - <div class="container-fluid"> + <div class="container-fluid container-fluid-top"> #**##else - <div class="container"> + <div class="container container-top"> #**##end <header> <div id="banner"> diff --git a/src/main/resources/css/maven-theme.css b/src/main/resources/css/maven-theme.css index 218ed82..63ac57c 100644 --- a/src/main/resources/css/maven-theme.css +++ b/src/main/resources/css/maven-theme.css @@ -81,10 +81,14 @@ li.none { } body.topBarEnabled { - padding-top: 60px; + /* calculated height of the topbar */ + padding-top: 43px; } -body.topBarDisabled { - padding-top: 20px; + +div.container-top, +div.container-fluid-top { + /* Bootstrap already adds 10 px to <h1> */ + padding-top: 10px; } .builtBy {
