https://bugs.freedesktop.org/show_bug.cgi?id=47957
Bug #: 47957
Summary: EasyHack: dung out laughable cui color options code
...
Classification: Unclassified
Product: LibreOffice
Version: LibO Master
Platform: Other
OS/Version: All
Status: UNCONFIRMED
Severity: normal
Priority: medium
Component: Libreoffice
AssignedTo: [email protected]
ReportedBy: [email protected]
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 !
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
