[Github-comments] [geany] Make it possible to define default symbol_list_sort_mode (#581)

Jiří Techet notifications at xxxxx
Sun Jan 10 17:57:16 UTC 2016


>
> @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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160110/0bcf8532/attachment.html>


More information about the Github-comments mailing list