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

<p>Looks good apart from the comments below</p><hr>

<p>In <a href="https://github.com/geany/geany/pull/1697#discussion_r304918533">src/sciwrappers.h</a>:</p>
<pre style='color:#555'>> @@ -82,6 +82,7 @@ gchar                              sci_get_char_at                         (ScintillaObject *sci, gint pos);
 void                           sci_scroll_caret                        (ScintillaObject *sci);
 gint                           sci_find_text                           (ScintillaObject *sci, gint flags, struct Sci_TextToFind *ttf);
 void                           sci_set_font                            (ScintillaObject *sci, gint style, const gchar *font, gint size);
+gint                           sci_get_fontsize                        (ScintillaObject *sci, gint style);
</pre>
<p>this should be in the <code>#ifdef GEANY_PRIVATE</code> section as it's not part of the API</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1697#discussion_r304941569">src/ui_utils.c</a>:</p>
<pre style='color:#555'>> @@ -305,6 +306,21 @@ static gchar *create_statusbar_statistics(GeanyDocument *doc,
                                g_string_append_printf(stats_str, "%d",
                                        sci_get_style_at(doc->editor->sci, pos));
                                break;
+                       case 'z':
+                               zoom = sci_get_zoom(doc->editor->sci);
+                               if (zoom != 0)
+                               {
+                                       gint size, percent;
+
+                                       size = sci_get_fontsize(doc->editor->sci, STYLE_DEFAULT);
+                                       if (size > 0)
+                                       {
+                                               percent = (size + zoom) * 100 / size;
+                                               g_string_append_c(stats_str, ' ');
</pre>
<p>this should probably use <code>sp</code> like other similar replacements above.<br>
It also probably should be <em>after</em> the actual contents (see for <code>%r</code> and <code>%m</code>). <code>%Y</code> seems to be a little off, and given it's not targeted to the average user and always displays something it's not a good example :)</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/1697?email_source=notifications&email_token=AAIOWJ5HHILE2TALRKF5QLDQAB4E7A5CNFSM4EFEY7TKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB63RJZA#pullrequestreview-263656676">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ26JBPAFOPCSGMDY6TQAB4E7ANCNFSM4EFEY7TA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ4A7CZFSNP5AORQPVLQAB4E7A5CNFSM4EFEY7TKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB63RJZA.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/pull/1697?email_source=notifications\u0026email_token=AAIOWJ5HHILE2TALRKF5QLDQAB4E7A5CNFSM4EFEY7TKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB63RJZA#pullrequestreview-263656676",
"url": "https://github.com/geany/geany/pull/1697?email_source=notifications\u0026email_token=AAIOWJ5HHILE2TALRKF5QLDQAB4E7A5CNFSM4EFEY7TKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB63RJZA#pullrequestreview-263656676",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>