If you fire up your editor and read:
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 !
Did you mean ColorConfigWindow_Impl? Or least both ColorConfigCtrl_Impl AND ColorConfigWindow_Impl?
Nevermind, I've been looking at the code more. I'll see what I can come up with.
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":
fdo#47957 refactoring and simplifying the color config settings
Migrating Whiteboard tags to Keywords: (EasyHack,DifficultyBeginner,SkillCpp,TopicCleanup)