Avoid inline css

Icinga · Aug 8, 2023 · 238f471 · 238f471
1 parent be1f56b
commit 238f471
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 17 deletions.
diff --git a/application/controllers/ProcessController.php b/application/controllers/ProcessController.php
@@ -182,8 +182,7 @@ protected function prepareControls($bp, $renderer)
                 'a',
                 [
                     'href'  => $this->url()->without('showFullscreen')->without('view'),
-                    'title' => $this->translate('Leave full screen and switch back to normal mode'),
-                    'style' => 'float: right'
+                    'title' => $this->translate('Leave full screen and switch back to normal mode')
                 ],
                 Html::tag('i', ['class' => 'icon icon-resize-small'])
             ));

diff --git a/application/forms/AddNodeForm.php b/application/forms/AddNodeForm.php
@@ -176,7 +176,6 @@ protected function selectHost()
             'label'        => $this->translate('Hosts'),
             'required'     => true,
             'size'         => 8,
-            'style'        => 'width: 25em',
             'multiOptions' => $this->enumHostList(),
             'description'  => $this->translate(
                 'Hosts that should be part of this business process node'
@@ -222,7 +221,6 @@ protected function addServicesElement($host)
             'label'        => $this->translate('Services'),
             'required'     => true,
             'size'         => 8,
-            'style'        => 'width: 25em',
             'multiOptions' => $this->enumServiceList($host),
             'description'  => $this->translate(
                 'Services that should be part of this business process node'
@@ -241,7 +239,6 @@ protected function addFilteredHostsElement($filter)
             'label'        => $this->translate('Hosts'),
             'required'     => true,
             'size'         => 8,
-            'style'        => 'width: 25em',
             'multiOptions' => $this->enumHostListByFilter($filter),
             'description'  => $this->translate(
                 'Hosts that should be part of this business process node'
@@ -260,7 +257,6 @@ protected function addFilteredServicesElement($filter)
             'label'        => $this->translate('Services'),
             'required'     => true,
             'size'         => 8,
-            'style'        => 'width: 25em',
             'multiOptions' => $this->enumServiceListByFilter($filter),
             'description'  => $this->translate(
                 'Services that should be part of this business process node'
@@ -362,7 +358,6 @@ protected function selectProcess()
                 'label'        => $this->translate('Process nodes'),
                 'required'     => true,
                 'size'         => 8,
-                'style'        => 'width: 25em',
                 'multiOptions' => $this->enumProcesses($file),
                 'description'  => $this->translate(
                     'Other processes that should be part of this business process node'

diff --git a/application/forms/EditNodeForm.php b/application/forms/EditNodeForm.php
@@ -262,7 +262,6 @@ protected function selectProcess()
             'label'        => $this->translate('Process nodes'),
             'required'     => true,
             'size'         => 8,
-            'style'        => 'width: 25em',
             'multiOptions' => $this->enumProcesses(),
             'description'  => $this->translate(
                 'Other processes that should be part of this business process node'

diff --git a/library/Businessprocess/Renderer/TileRenderer/NodeTile.php b/library/Businessprocess/Renderer/TileRenderer/NodeTile.php
@@ -105,12 +105,9 @@ public function render()
             $this->add($link);
         } else {
             $this->add(Html::tag(
-                'a',
-                Html::tag(
-                    'span',
-                    ['style' => 'font-size: 75%'],
-                    sprintf('Trying to access a missing business process node "%s"', $node->getNodeName())
-                )
+                'span',
+                ['class' => 'missing-node-msg'],
+                sprintf('Trying to access a missing business process node "%s"', $node->getNodeName())
             ));
         }
 

diff --git a/public/css/module.less b/public/css/module.less
@@ -77,8 +77,11 @@ a:focus {
     }
 }
 
-.controls .sort-control {
-    float: right;
+.controls {
+    &.sort-control,
+    &.want-fullscreen > a {
+        float: right;
+    }
 }
 
 form a {
@@ -471,7 +474,8 @@ td > a > .state-badges {
         border: 1px solid @body-bg-color;
     }
 
-    > a {
+    > a,
+    .missing-node-msg {
         display: block;
         text-decoration: none;
         font-size: 0.7em;
@@ -481,6 +485,10 @@ td > a > .state-badges {
         word-wrap: break-word;
     }
 
+    .missing-node-msg {
+        font-size: 0.5em;
+    }
+
     &:hover {
         box-shadow: 0 0 .2em @gray;
     }