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