From 174b098724e15b81cca3fde14db0fec27b23e321 Mon Sep 17 00:00:00 2001 From: chris moran Date: Tue, 30 Jun 2020 07:54:51 -0400 Subject: [PATCH] loop/opt/alt/etc titles should always wrap --- src/diagrams/sequence/sequenceRenderer.js | 23 ++++++++++------------- 1 file changed, 10 insertions(+), 13 deletions(-) diff --git a/src/diagrams/sequence/sequenceRenderer.js b/src/diagrams/sequence/sequenceRenderer.js index cbf31252c..60be03f6a 100644 --- a/src/diagrams/sequence/sequenceRenderer.js +++ b/src/diagrams/sequence/sequenceRenderer.js @@ -459,20 +459,17 @@ function adjustLoopHeightForWrap(loopWidths, msg, preMargin, postMargin, addLoop bounds.bumpVerticalPos(preMargin); let heightAdjust = postMargin; if (msg.id && msg.message && loopWidths[msg.id]) { - if (msg.wrap) { - let loopWidth = loopWidths[msg.id].width; - let textConf = conf.messageFont(); - msg.message = utils.wrapLabel(`[${msg.message}]`, loopWidth - 2 * conf.wrapPadding, textConf); - msg.width = loopWidth; + let loopWidth = loopWidths[msg.id].width; + let textConf = conf.messageFont(); + msg.message = utils.wrapLabel(`[${msg.message}]`, loopWidth - 2 * conf.wrapPadding, textConf); + msg.width = loopWidth; + msg.wrap = true; - // const lines = msg.message.split(common.lineBreakRegex).length; - const textDims = utils.calculateTextDimensions(msg.message, textConf); - const totalOffset = textDims.height - conf.labelBoxHeight; - heightAdjust = postMargin + totalOffset; - logger.debug(`${totalOffset} - ${msg.message}`); - } else { - msg.message = `[${msg.message}]`; - } + // const lines = msg.message.split(common.lineBreakRegex).length; + const textDims = utils.calculateTextDimensions(msg.message, textConf); + const totalOffset = textDims.height - conf.labelBoxHeight; + heightAdjust = postMargin + totalOffset; + logger.debug(`${totalOffset} - ${msg.message}`); } addLoopFn(msg); bounds.bumpVerticalPos(heightAdjust);