Missed line break?

@lowercasedonky @ezsh

Definitely seems we are missing a line break and I need to call it a night if either of you want to track down what got missed.

This is what it should be.

How to reproduce that?

How to reproduce that?

Load a save, then go to the SummaryOptions and set Mental stats to Abbreviate.
I have fixed it in testing by appending a <br> tag to <span id="slave-${slaves[_ssi].ID}">&nbsp;</span> (line 5618 of slaveSummaryWidgets.js).

Full text:
Before:
After:

Abrv text:
Before
After:

It happens when images are disabled.

I think the proper correction would be to terminate the "transfer to" list with a <br>:

diff --git a/src/js/slaveSummaryWidgets.js b/src/js/slaveSummaryWidgets.js
index 688f7a464..f8333ae07 100644
--- a/src/js/slaveSummaryWidgets.js
+++ b/src/js/slaveSummaryWidgets.js
@@ -5609,7 +5609,7 @@ App.UI.slaveSummaryList = function (passageName) {
            }
          } /* Closes transfer options check */
 
-         res.push(transfers.join(' | '));
+         res.push(transfers.join('&thinsp;|&thinsp;') + '<br>');
        } /* CLOSES FUCKDOLL CHECK */
      }
    }/* closes _numFacilities */

I think the proper correction would be to terminate the "transfer to" list with...

Nope, then it renders an empty line when image art is enabled.

This change seems does the trick:

diff --git a/src/js/slaveSummaryWidgets.js b/src/js/slaveSummaryWidgets.js
index 688f7a464..69cbc5998 100644
--- a/src/js/slaveSummaryWidgets.js
+++ b/src/js/slaveSummaryWidgets.js
@@ -5617,9 +5617,10 @@ App.UI.slaveSummaryList = function (passageName) {
    if ((passageName != 'Main') || (V.Flag !== undefined) || (V.useSlaveSummaryTabs == 0) || (tabName == "all")) {
      res.push(`<span id="slave-${slaves[_ssi].ID}">&nbsp;</span>`);
    }
+   res.push('<br/>');
 
    if (slaveImagePrinted) {
-     res.push('<br/>&nbsp;&nbsp;&nbsp;&nbsp;')
+     res.push('&nbsp;&nbsp;&nbsp;&nbsp;')
    }
 
    clearSummaryCache();

closed via commit e9806f26