><br>
> @tchee There is a bug in the prefs UI: when deactivating "show sidebar",<br>
> the new option isn't grayed out. What I'd do is make the whole vbox46<br>
> insensitive in on_sidebar_visible_toggled() -- and renamed the glade ID<br>
> to something more sensible, like i.e. box_sidebar_visible_children or<br>
> alike.<br>
><br>
Sounds good, thanks for noticing.<br>
<br>
<br>
<br>
> ------------------------------<br>
><br>
> Also, what about using an multi-choice UI like this (or a drop-down, but<br>
> well) instead of a checkbox?:<br>
> [image: capture]<br>
> <https://cloud.githubusercontent.com/assets/793526/12222199/63be1498-b7b5-11e5-88fe-7278518c50bb.png><br>
> This would<br>
><br>
>    - make it easier for the user to see the available options<br>
>    - make it easier to add new options in the future (although unlikely)<br>
><br>
> Yeah, looks nicer (I'd prefer the radios instead the drop-down). What<br>
about using "Default sort mode:" or "Initial sort mode:" for the label? It<br>
might make it clearer the settings is applied only upon file opening but<br>
not if the settings is changed later.<br>
<br>
<br>
><br>
><br>
> And while at it, what about storing the actual sort mode in the prefs<br>
> instead of a flag? Again, scales better.<br>
><br>
Sounds good too.<br>
<br>
<br>
> If we want the above, I have some working version I could polish and<br>
> publish.<br>
><br>
Great, will have a look at your changes.<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany/pull/581#issuecomment-170374561">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ96BLqJxe1mpRE9K4YVNYgpnYYjlks5pYpL8gaJpZM4FbTe1.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/581#issuecomment-170374561"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>