<p><b>@b4n</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1554#discussion_r139496331">src/editor.c</a>:</p>
<pre style='color:#555'>> @@ -2394,7 +2394,7 @@ typedef struct
 } SelectionRange;
 
 
-#define CURSOR_PLACEHOLDER "_" /* Would rather use … but not all docs are unicode */
+#define CURSOR_PLACEHOLDER "..." /* Would rather use … but not all docs are unicode */
</pre>
<p>I guess yeah ideally the placeholder wouldn't be any actual character but simply a display thing, but I'm not sure Scintilla allows for this, does it?  It would need be able to have an indicator that takes more room than the range it's on, which doesn't sounds likely, but I might miss an existing feature.</p>
<p>Otherwise for what the placeholder is I don't care that much; I just didn't feel like <code>_</code> looked straightforward to me when using the thing, but I don't have a string opinion either.  I still feel like <a href="https://github.com/geany/geany/commit/957b49b868214a4aea81641dedeabb98a497e4c1#commitcomment-23346900" class="commit-link"><tt>957b49b</tt>#commitcomment-23346900</a>, but if the consensus is to keep <code>_</code> I won't argue.</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/pull/1554#discussion_r139496331">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ8KBRr-gxNREVs1d2NjpY43P3thtks5sjrBogaJpZM4OgvkT">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ0SmQm77jNNXIDBD4yKCVFev76-cks5sjrBogaJpZM4OgvkT.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1554#discussion_r139496331"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@b4n commented on #1554"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1554#discussion_r139496331"}}}</script>