If you fire up your editor and read: cui/source/options/optcolor.cxx ... This is the tools->options->appearance widget. At a minimum it is necessary to laugh at: ColorConfigCtrl_Impl(this, CUI_RES( CT_COLORCONFIG ) )) - and the hideous member functions, and laborious code in there. We should have a little loop to construct the array of widget pointers on the heap: new FixedText(...); etc. We should use a struct to store each of the three elements, instead of multiple separate arrays, we should simplify all the logic in this big heap. It'd also be a nice idea (as a 2nd, optional thing) to re-use a standard widget if there is one, instead of this custom, hacked-up, manually scrolled mess :-) Thanks for dunging it out !
Hi, Michael, Did you mean ColorConfigWindow_Impl? Or least both ColorConfigCtrl_Impl AND ColorConfigWindow_Impl? http://opengrok.libreoffice.org/xref/core/cui/source/options/optcolor.cxx Thanks.
Hi, Michael, Nevermind, I've been looking at the code more. I'll see what I can come up with.
Hi, Sorry, I got distracted with other EasyHacks. I'm setting this back to new in case someone else wants to work on it before I get back to it.
*** Bug 48919 has been marked as a duplicate of this bug. ***
Deleted "Easyhack" from summary.
*** Bug 51204 has been marked as a duplicate of this bug. ***
Uray M. Janos committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=2ca73bfbf46332bcd52619e2e82e6a2a744eaa8c fdo#47957 refactoring and simplifying the color config settings
Migrating Whiteboard tags to Keywords: (EasyHack,DifficultyBeginner,SkillCpp,TopicCleanup) [NinjaEdit]