<p>Don't agree with using the theme (at least as proposed above).</p>
<p>While at first glance it might seem nice to have the themes provide the styling, in reality they won't provide a <code>Geany-compiler-error-colour</code> or equivalent.  So we would have to use standard colours if it was to follow the theme.</p>
<p>Unfortunately standard warning and error colours are, as <a href="https://github.com/codebrainz" class="user-mention">@codebrainz</a> said, designed for large slabs of colour, and not for fine text, so they may not work in that context.  In fact my <a href="https://github.com/linuxmint/mint-y-theme/blob/d1bdb934f1c7a7e709d134f1dcdcae3faba2f8a3/src/gtk-3.0/3.18/gtk-dark.css#L2346">dark theme</a> sets warning colour to black, NSFW for dark and warning you say, but it then changes the background to a  lighter warning or error colour, thus showing it in a larger slab.  So Geany would have to be changed to set both the colour and the row background in the treeview rows to use this in Geany, although it will be pretty intrusive on a dark theme to have brightly coloured row backgrounds in the compiler window..</p>
<p>So the user is left having to learn how to edit a local modification to themes in either GTKrc or GTK-not-quite-really-CSS, depending on the version of GTK in use, instead of a file format controlled by Geany and so not likely to change.</p>
<p>I can confirm the problem, having only recently switched to a dark theme and the distro being Mint, its greenish, so red text is difficult to read against the background (and my colour acuity is (just) classified as safe, pity the 10% of males with real colour blindness).</p>
<p>So since it is a real problem (I have it <g-emoji alias="smile" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f604.png" ios-version="6.0">😄</g-emoji> ) I propose that we wait for a bit and see if a viable themes based pull request emerges, if not we commit this fairly simple solution as an interim (assuming it works, not tested yet) with an issue to change it to using themes.</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/1377#issuecomment-275287045">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJyiKxnBdsmZu036l9uHz0zEgbQAkks5rV_lygaJpZM4LtI7n">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ4zQESfuNPOySWfVLWFw0lut6Jutks5rV_lygaJpZM4LtI7n.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/1377#issuecomment-275287045"></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":"@elextr in #1377: Don't agree with using the theme (at least as proposed above).\r\n\r\nWhile at first glance it might seem nice to have the themes provide the styling, in reality they won't provide a `Geany-compiler-error-colour` or equivalent.  So we would have to use standard colours if it was to follow the theme.\r\n\r\nUnfortunately standard warning and error colours are, as @codebrainz said, designed for large slabs of colour, and not for fine text, so they may not work in that context.  In fact my [dark theme](https://github.com/linuxmint/mint-y-theme/blob/d1bdb934f1c7a7e709d134f1dcdcae3faba2f8a3/src/gtk-3.0/3.18/gtk-dark.css#L2346) sets warning colour to black, NSFW for dark and warning you say, but it then changes the background to a  lighter warning or error colour, thus showing it in a larger slab.  So Geany would have to be changed to set both the colour and the row background in the treeview rows to use this in Geany, although it will be pretty intrusive on a dark theme to have brightly coloured row backgrounds in the compiler window..\r\n\r\nSo the user is left having to learn how to edit a local modification to themes in either GTKrc or GTK-not-quite-really-CSS, depending on the version of GTK in use, instead of a file format controlled by Geany and so not likely to change.\r\n\r\nI can confirm the problem, having only recently switched to a dark theme and the distro being Mint, its greenish, so red text is difficult to read against the background (and my colour acuity is (just) classified as safe, pity the 10% of males with real colour blindness).\r\n\r\nSo since it is a real problem (I have it :smile: ) I propose that we wait for a bit and see if a viable themes based pull request emerges, if not we commit this fairly simple solution as an interim (assuming it works, not tested yet) with an issue to change it to using themes. "}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1377#issuecomment-275287045"}}}</script>