mbien commented on PR #7616: URL: https://github.com/apache/netbeans/pull/7616#issuecomment-2255880231
adding a quick way to change sorting (and similar setttings) is certainly useful, since this might change from usecase to usecase. I am a bit skeptical of the purpose of linking to the coloring options though. This is something what is done once (if at all) and then no longer really relevant during dev time. Highlighting is a generic font/colors attribute. Sometimes showing less is better :) (I also do agree with @matthiasblaesing, the coloring setting should probably not even exist since it should be under fonts/colors - but this is not your fault, the UI was already there) Now, if we agree that we don't care about coloring options, all what is left is the order-by combo box. And since it has only two items, it is a checkbox in disguise, which would fit pretty well right next to the other check boxes ;). But if you implemented the link to the options already - thats also fine with me. There is a problem though: duplicated code. There is an almost identical dialog for types (ctrl+o), which likely going to need that setting too (I hope "Go to Dialog Settings" do in fact apply to both dialogs). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
