From b167b415a3359c90c673a3a6e598e897670dbc1f Mon Sep 17 00:00:00 2001 From: Jake Dallimore Date: Tue, 17 Jan 2017 13:13:08 +0800 Subject: [PATCH] MDL-57287 themes: fix buttons in default dashboard page header This fix modifies styling for pages having no navbar, such as the dashboard, site home and default dashboard pages and fixes an overlap problem with the buttons and context header. --- theme/boost/classes/output/core_renderer.php | 8 +++++--- theme/boost/scss/moodle/core.scss | 6 ++++-- theme/boost/scss/moodle/modules.scss | 7 ------- 3 files changed, 9 insertions(+), 12 deletions(-) diff --git a/theme/boost/classes/output/core_renderer.php b/theme/boost/classes/output/core_renderer.php index 432c6272a68..8787a344f40 100644 --- a/theme/boost/classes/output/core_renderer.php +++ b/theme/boost/classes/output/core_renderer.php @@ -78,15 +78,17 @@ class core_renderer extends \core_renderer { $html .= html_writer::start_div('card'); $html .= html_writer::start_div('card-block'); $html .= html_writer::div($this->context_header_settings_menu(), 'pull-xs-right context-header-settings-menu'); + $html .= html_writer::start_div('pull-xs-left'); $html .= $this->context_header(); + $html .= html_writer::end_div(); $pageheadingbutton = $this->page_heading_button(); if (empty($PAGE->layout_options['nonavbar'])) { - $html .= html_writer::start_div('clearfix', array('id' => 'page-navbar')); + $html .= html_writer::start_div('clearfix w-100 pull-xs-left', array('id' => 'page-navbar')); $html .= html_writer::tag('div', $this->navbar(), array('class' => 'breadcrumb-nav')); - $html .= html_writer::div($pageheadingbutton, 'breadcrumb-button'); + $html .= html_writer::div($pageheadingbutton, 'breadcrumb-button pull-xs-right'); $html .= html_writer::end_div(); } else if ($pageheadingbutton) { - $html .= html_writer::div($pageheadingbutton, 'breadcrumb-button nonavbar'); + $html .= html_writer::div($pageheadingbutton, 'breadcrumb-button nonavbar pull-xs-right'); } $html .= html_writer::tag('div', $this->course_header(), array('id' => 'course-header')); $html .= html_writer::end_div(); diff --git a/theme/boost/scss/moodle/core.scss b/theme/boost/scss/moodle/core.scss index c6690751eaa..cfa91469252 100644 --- a/theme/boost/scss/moodle/core.scss +++ b/theme/boost/scss/moodle/core.scss @@ -9,9 +9,11 @@ .context-header-settings-menu, .region-main-settings-menu { float: right; - width: 4em; + width: auto; + max-width: 4em; height: 2em; display: block; + margin-top: 4px; } .context-header-settings-menu .dropdown-toggle > .icon, @@ -1782,12 +1784,12 @@ header { .page-header-image, .page-header-headings { + float: left; display: block; position: relative; } .page-header-image { - float: left; margin-right: 1em; margin-bottom: 1em; } diff --git a/theme/boost/scss/moodle/modules.scss b/theme/boost/scss/moodle/modules.scss index 9e27d2bcb7d..a7ea7558473 100644 --- a/theme/boost/scss/moodle/modules.scss +++ b/theme/boost/scss/moodle/modules.scss @@ -192,14 +192,7 @@ select { } .breadcrumb-button { - float: right; margin-top: 4px; - - @include media-breakpoint-up('md') { - &.nonavbar { - margin-top: -3rem; - } - } } .breadcrumb-button .singlebutton {