<p>For</p>
<p>abcde;</p>
<p>simple 't;' does the right thing and places the cursor on top of character<br>
'e' just before ';'. However, for composed commands such as 'dt;', vim<br>
also deletes the character 'e' which vimode currently doesn't.</p>
<p>Vim behaves in a bit inconsistent way regarding this behavior - while<br>
for 'dw' (delete all until next word) it doesn't delete the first character<br>
of the next word onto which the cursor is placed, for 'de' (delete until<br>
next word end) it includes the final character of the deleted word. This<br>
behavior makes sense from the use case point of view but makes handling<br>
of such situations inconsistent.</p>
<p>This patch lists functions which behave this way and includes the<br>
destination character to the selection.</p>
<p><span class="issue-keyword tooltipped tooltipped-se" aria-label="This pull request closes issue #1052.">Fixes</span> <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="775309877" data-permission-text="Title is private" data-url="https://github.com/geany/geany-plugins/issues/1052" data-hovercard-type="issue" data-hovercard-url="/geany/geany-plugins/issues/1052/hovercard" href="https://github.com/geany/geany-plugins/issues/1052">#1052</a>.</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/geany/geany-plugins/pull/1104'>https://github.com/geany/geany-plugins/pull/1104</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>vimode: Add special handling to include destination char for some commands</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/geany/geany-plugins/pull/1104/files#diff-34066c45b9c50afd85870b2a1eaf06df2531aa3dda19ac94bb099212566ee24d">vimode/src/cmd-runner.c</a>
    (24)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/geany/geany-plugins/pull/1104.patch'>https://github.com/geany/geany-plugins/pull/1104.patch</a></li>
  <li><a href='https://github.com/geany/geany-plugins/pull/1104.diff'>https://github.com/geany/geany-plugins/pull/1104.diff</a></li>
</ul>

<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-plugins/pull/1104">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ5NSF3JK7AQVS63SL3T7Z745ANCNFSM5DHH2QSA">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<img src="https://github.com/notifications/beacon/AAIOWJ5UNXPDORHRIAMR2SLT7Z745A5CNFSM5DHH2QSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4OV6Z3DQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany-plugins/pull/1104",
"url": "https://github.com/geany/geany-plugins/pull/1104",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>