commontk/CTK — comment on commit 4314c15836b87f2946665bda7e442ecd4593cb16
File: Libs/Widgets/ctkToolTipTrapper.h line 67
By finetjul:

I don't think there is enough "feature" to split the class in 2. I believe 1 class that contains everything that is tooltip related is good enough.
Every object can always be split into smaller objects, but the line has to be drawn somewhere. What if you want to control the delay, color, position, behavior, animation,... of tooltips ? You would create 1 class for each property ? You would then need a tooltip manager to control all of those properties because you don't want the application developer to manually instanciate all those classes. Why not have this ctkToolTipTrapper class be directly the "manager" of tooltip properties (display of tooltips: visible/hidden, plaintext/richtext...) and keep all of the work in 1 class ? Having separate classes sounds a bit over-engineered to me, especially when the logic of those can be written in 1 line of code (hide tooltip, convert to richtext)
The name ctkToolTipTrapper might sound a bit too specialized and it could be renamed into something more generic (ctkToolTip, ctkToolTipManager, ...).