<p>I actually didn't realize that there was already the option to customize some colors using the GTK theme, which is why I did it using colorschemes.</p>
<p>I agree that using the GTK theme would be better, as it is more consistent with how geany works at the moment (-> as codebrainz pointed out: colorschemes for the editor, gtkrc for the other widgets). While it is true that no theme will provide the colors for those specific styles, this is also the case for all the other widgets that are at styled using geany.gtkrc (tab foregrounds, etc).</p>
<p>Additionally, the current implementation only changes the foreground color according to the colorscheme, whereas the background color is still defined by GTK, meaning that using a dark colorscheme and a light GTK theme or vice versa will again lead to unreadable messages.</p>
<p>I changed the fix to load the colors from the GTK theme, should I make a new pull request for it or update this one?</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-275708703">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ1y8XOfx7RFzqqQ-LPtniTis0Zruks5rWhvXgaJpZM4LtI7n">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ0YgKZ-XNstwk1PXR1Gk3D5mVZptks5rWhvXgaJpZM4LtI7n.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-275708703"></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":"@Rakksor in #1377: I actually didn't realize that there was already the option to customize some colors using the GTK theme, which is why I did it using colorschemes. \r\n\r\nI agree that using the GTK theme would be better, as it is more consistent with how geany works at the moment (-\u003e as codebrainz pointed out: colorschemes for the editor, gtkrc for the other widgets). While it is true that no theme will provide the colors for those specific styles, this is also the case for all the other widgets that are at styled using geany.gtkrc (tab foregrounds, etc).\r\n\r\nAdditionally, the current implementation only changes the foreground color according to the colorscheme, whereas the background color is still defined by GTK, meaning that using a dark colorscheme and a light GTK theme or vice versa will again lead to unreadable messages.\r\n\r\nI changed the fix to load the colors from the GTK theme, should I make a new pull request for it or update this one?"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1377#issuecomment-275708703"}}}</script>