From 64bff8a106d458c0932ff2863e32259e8eb58137 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?David=20Mudr=C3=A1k?= Date: Fri, 10 Apr 2020 14:03:23 +0200 Subject: [PATCH] MDL-68393 admin: Fix some minor styling issues on the notifications page The patch makes the available update information elements more distinguishable, the Download button more prominent, and the Check for available update button better aligned. It reduces the space between multiple available Moodle updates, too. --- admin/renderer.php | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/admin/renderer.php b/admin/renderer.php index daaf68466b1..d2ac54e793e 100644 --- a/admin/renderer.php +++ b/admin/renderer.php @@ -823,7 +823,7 @@ class core_admin_renderer extends plugin_renderer_base { } } - $updateinfo .= $this->container_start('checkforupdates'); + $updateinfo .= $this->container_start('checkforupdates mt-1'); $fetchurl = new moodle_url('/admin/index.php', array('fetchupdates' => 1, 'sesskey' => sesskey(), 'cache' => 0)); $updateinfo .= $this->single_button($fetchurl, get_string('checkforupdates', 'core_plugin')); if ($fetch) { @@ -912,7 +912,7 @@ class core_admin_renderer extends plugin_renderer_base { */ protected function moodle_available_update_info(\core\update\info $updateinfo) { - $boxclasses = 'moodleupdateinfo'; + $boxclasses = 'moodleupdateinfo mb-2'; $info = array(); if (isset($updateinfo->release)) { @@ -932,7 +932,8 @@ class core_admin_renderer extends plugin_renderer_base { } if (isset($updateinfo->download)) { - $info[] = html_writer::link($updateinfo->download, get_string('download'), array('class' => 'info download')); + $info[] = html_writer::link($updateinfo->download, get_string('download'), + array('class' => 'info download btn btn-secondary')); } if (isset($updateinfo->url)) { @@ -940,9 +941,9 @@ class core_admin_renderer extends plugin_renderer_base { array('class' => 'info more')); } - $box = $this->output->box_start($boxclasses); - $box .= $this->output->box(implode(html_writer::tag('span', ' ', array('class' => 'separator')), $info), ''); - $box .= $this->output->box_end(); + $box = $this->output->container_start($boxclasses); + $box .= $this->output->container(implode(html_writer::tag('span', ' | ', array('class' => 'separator')), $info), ''); + $box .= $this->output->container_end(); return $box; }