FIX Moving the version tag to below the action buttons

bringyourownideas · May 23, 2018 · 9b01050 · 9b01050
1 parent ef38d2e
commit 9b01050
Show file tree

Hide file tree

Showing 4 changed files with 62 additions and 16 deletions.
diff --git a/css/sitesummary.css b/css/sitesummary.css
@@ -2,6 +2,11 @@
   display:block;
 }
 
+.cms .ss-gridfield > div.site-summary__clearfix {
+    margin: 0;
+    clear: both;
+}
+
 .gridfield-button-link {
     margin-bottom: 12px;
 }
diff --git a/src/Forms/GridFieldHtmlFragment.php b/src/Forms/GridFieldHtmlFragment.php
@@ -0,0 +1,42 @@
+<?php
+
+/**
+ * Facilitates adding arbitrary HTML to grid fields
+ *
+ * @package forms
+ * @subpackage fields-gridfield
+ */
+
+class GridFieldHtmlFragment implements GridField_HTMLProvider
+{
+    /**
+     * Fragment to write the html fragment to.
+     * @var string
+     */
+    protected $targetFragment;
+
+    /**
+     * An HTML fragment to render
+     * @var string
+     */
+    protected $htmlFragment;
+
+    /**
+     * @param string $targetFragment Fragment to write the html fragment to.
+     * @param string $htmlFragment An HTML fragment to render
+     */
+    public function __construct($targetFragment, $htmlFragment)
+    {
+        $this->targetFragment = $targetFragment;
+        $this->htmlFragment = $htmlFragment;
+    }
+
+    /**
+     * @param GridField $gridField
+     * @return array
+     */
+    public function getHTMLFragments($gridField)
+    {
+        return [$this->targetFragment => $this->htmlFragment];
+    }
+}
diff --git a/src/Reports/SiteSummary.php b/src/Reports/SiteSummary.php
@@ -12,20 +12,6 @@ public function title()
         return _t(__CLASS__ . '.TITLE', 'Installed modules');
     }
 
-    public function getCMSFields()
-    {
-        $fields = parent::getCMSFields();
-
-        $this->beforeExtending('updateCMSFields', function (FieldList $fields) {
-            $fields->unshift(new LiteralField(
-                'Version',
-                '<h3>' . _t(__CLASS__ . '.VERSION', 'Version: ') . $this->resolveCmsVersion() . '</h3>'
-            ));
-        });
-
-        return $fields;
-    }
-
     public function sourceRecords()
     {
         $packageList = Package::get();
@@ -58,9 +44,20 @@ public function getReportField()
         Requirements::css('silverstripe-maintenance/css/sitesummary.css');
         $grid = parent::getReportField();
         $config = $grid->getConfig();
+
+        $versionHtml = ArrayData::create([
+            'Title' => _t(__CLASS__ . '.VERSION', 'Version'),
+            'Version' => $this->resolveCmsVersion(),
+        ])->renderWith('SiteSummary_VersionHeader');
+
         $config->addComponents(
-            Injector::inst()->create('GridFieldButtonRow', 'before'),
-            Injector::inst()->create('GridFieldLinkButton', 'https://addons.silverstripe.org', 'buttons-before-left')
+            Injector::inst()->create(GridFieldButtonRow::class, 'before'),
+            Injector::inst()->create(
+                GridFieldLinkButton::class,
+                'https://addons.silverstripe.org',
+                'buttons-before-left'
+            ),
+            Injector::inst()->create(GridFieldHtmlFragment::class, 'header', $versionHtml)
         )
             ->getComponentByType(GridFieldExportButton::class)
             ->setExportColumns(

diff --git a/templates/SiteSummary_VersionHeader.ss b/templates/SiteSummary_VersionHeader.ss
@@ -0,0 +1,2 @@
+<div class="site-summary__clearfix"></div>
+<h3>{$Title.XML}: {$Version.XML}</h3>