From 464cb48a5cbba2d7050f0f7eb514dc97c97692f0 Mon Sep 17 00:00:00 2001
From: ezsh <ezsh.junk@gmail.com>
Date: Fri, 29 May 2020 16:05:22 +0200
Subject: [PATCH] Fix text wrapping for slave summary paragraphs
Replace CSS margin with explicit whitespace to allow line wrap between
consequent spans used in the abbreviated mode.
Contributes to #1823.
---
src/003-assets/CSS/slaveList.css | 5 -----
src/js/slaveSummaryHelpers.js | 3 +--
2 files changed, 1 insertion(+), 7 deletions(-)
diff --git a/src/003-assets/CSS/slaveList.css b/src/003-assets/CSS/slaveList.css
index cee97eac754..dab49048970 100644
--- a/src/003-assets/CSS/slaveList.css
+++ b/src/003-assets/CSS/slaveList.css
@@ -15,11 +15,6 @@ span.freeAssignment {
font-weight: bold;
}
-/* Slave Summary Item */
-.ssi {
- margin-right: 0.25em;
-}
-
/* Slave Summary Block */
.ssb {
margin-right: 2em;
diff --git a/src/js/slaveSummaryHelpers.js b/src/js/slaveSummaryHelpers.js
index 19630f896b9..9cf5433d3d6 100644
--- a/src/js/slaveSummaryHelpers.js
+++ b/src/js/slaveSummaryHelpers.js
@@ -31,12 +31,11 @@ App.UI.SlaveSummaryImpl = function() {
*/
function makeSpan(container, text, classNames, stdDecor = false, value) {
let r = document.createElement("span");
- r.classList.add("ssi");
_addClassNames(r, classNames);
if (value != undefined && V.summaryStats) { /* eslint-disable-line eqeqeq */
text += `[${value}]`;
}
- r.textContent = stdDecor ? `${capFirstChar(text)}.` : text;
+ r.textContent = stdDecor ? `${capFirstChar(text)}. ` : text + ' ';
if (container) {
container.appendChild(r);
}
--
GitLab