<blockquote>
<p>But with disabled line breaking, manual reflow limits lines to 69 characters.</p>
</blockquote>
<p>Yes because its using a setting that has a value that is one smaller.</p>
<blockquote>
<p>As a user, I expect Reflow with disabled Line Breaking to wrap lines <em>longer than</em> the marker's X as well, but instead it wraps <em>at</em> X. That's rather strange and inconvenient.</p>
</blockquote>
<p>I have to say I find it simple and consistent, line breaking always behaves the same, it breaks <strong>at</strong> the specified column, always, no matter what, thats being consistent.  Long line functionality marks <strong>after</strong> the specified column.</p>
<p>Note the two different emphasised words in the paragraph above, the two functions are different so of course they don't align.</p>
<p>Line breaking is a per document setting, it can be turned on and off for any document, but the settings are session wide.  Suddenly having manual reflow happen at a different place because automatic line breaking is turned on for one document and off for another, even when the settings are the same number would be confusing and wrong.</p>
<p>Perhaps reflow should only ever use the line breaking setting, even if automatic breaking is off.  That way the two functions are totally independent and internally consistent with themselves, instead of the same number (long line) being used in two different ways by two different functions.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/issues/2078#issuecomment-460648377">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ2ieudOKwJHuhTVnJl6VpCUcecZwks5vKY8igaJpZM4ag2kH">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ0499A8GdAmzpGv7dskjpIKmnDb_ks5vKY8igaJpZM4ag2kH.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #2078: \u003e But with disabled line breaking, manual reflow limits lines to 69 characters.\r\n\r\nYes because its using a setting that has a value that is one smaller.\r\n\r\n\u003e As a user, I expect Reflow with disabled Line Breaking to wrap lines _longer than_ the marker's X as well, but instead it wraps _at_ X. That's rather strange and inconvenient.\r\n\r\nI have to say I find it simple and consistent, line breaking always behaves the same, it breaks __at__ the specified column, always, no matter what, thats being consistent.  Long line functionality marks __after__ the specified column.\r\n\r\nNote the two different emphasised words in the paragraph above, the two functions are different so of course they don't align.\r\n\r\nLine breaking is a per document setting, it can be turned on and off for any document, but the settings are session wide.  Suddenly having manual reflow happen at a different place because automatic line breaking is turned on for one document and off for another, even when the settings are the same number would be confusing and wrong.\r\n\r\nPerhaps reflow should only ever use the line breaking setting, even if automatic breaking is off.  That way the two functions are totally independent and internally consistent with themselves, instead of the same number (long line) being used in two different ways by two different functions."}],"action":{"name":"View Issue","url":"https://github.com/geany/geany/issues/2078#issuecomment-460648377"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/issues/2078#issuecomment-460648377",
"url": "https://github.com/geany/geany/issues/2078#issuecomment-460648377",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>