> > @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
