Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

MDL-42184 remove the body top padding for bootstrapbase popup + fix g…

…et_device_type call

Conflicts:
	theme/bootstrapbase/style/moodle.css
  • Loading branch information...
commit bb12a401ecdd473c1c18353e7706128ff02f1a9c 1 parent ccbd4e4
@mouneyrac mouneyrac authored samhemelryk committed
View
19 theme/bootstrapbase/layout/popup.php
@@ -23,20 +23,23 @@
<meta name="viewport" content="width=device-width, initial-scale=1.0">
</head>
-<body <?php echo $OUTPUT->body_attributes(); ?>>
-
-<?php echo $OUTPUT->standard_top_of_body_html() ?>
-
<?php
- // If on desktop, then hide the header/footer.
- $hideclass = '';
- $devicetype = core_useragent::get_device_type();
- if($devicetype !== 'mobile' and $devicetype !== 'tablet') {
+// If on desktop, then hide the header/footer.
+$hideclass = '';
+$bodynoheader = '';
+$devicetype = get_device_type();
+if($devicetype !== 'mobile' and $devicetype !== 'tablet') {
// We can not use the Bootstrap responsive css classes because popups are phone sized on desktop.
$hideclass = 'hide';
+ $bodynoheader = 'bodynoheader';
}
?>
+<body <?php echo $OUTPUT->body_attributes(array($bodynoheader)); ?>>
+
+<?php echo $OUTPUT->standard_top_of_body_html() ?>
+
+
<header role="banner" class="navbar navbar-fixed-top moodle-has-zindex <?php echo $hideclass; ?>">
<nav role="navigation" class="navbar-inner">
<div class="container-fluid">
View
4 theme/bootstrapbase/less/moodle/core.less
@@ -1936,3 +1936,7 @@ img#persona_signin {
.breadcrumb-nav .breadcrumb {
margin: 0;
}
+
+body.bodynoheader {
+ padding-top: 0;
+}
View
2  theme/bootstrapbase/style/moodle.css
1 addition, 1 deletion not shown
Please sign in to comment.
Something went wrong with that request. Please try again.