<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Hi All!</span><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Last week I made small refinemets to the UI.</div>
<div style="font-family:arial,sans-serif;font-size:13px">- Selecting a color from the color picker popup now applies it immediately.</div><div style="font-family:arial,sans-serif;font-size:13px">- Loading soc palettes is now done lazily.</div>
<div style="font-family:arial,sans-serif;font-size:13px">- I changed a vector of pointer to a ptr_vector.</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">
I was also investigating the possibility of killing some classes.</div><div style><font face="arial, sans-serif"><b>SdrShadowColorItem</b>, </font><b>XFillColorItem</b>, <b>XFormTextShadowColorItem</b>, <b>XLineColorItem</b> and <b>XSecondaryFillColorItem</b> all inherit from <b>XColorItem</b>, but they don't really add anything, just have a different <b>nWhich</b> value. I considered replacing the call sites of the ctors of the child classes with ctors for the parent class, but with the different <b>nWhich</b> value as a parameter, but that'd lose type checking, so I decided against it.</div>
<div style><br></div><div style>My other problem is, I don't know how I could replace <b style="font-size:13px;font-family:arial,sans-serif">SvxLineColorToolBoxControl</b><span style="font-size:13px;font-family:arial,sans-serif"> with </span><b style="font-size:13px;font-family:arial,sans-serif">SvxColorToolBoxControl</b><span style="font-size:13px;font-family:arial,sans-serif">. They were almost identical but now I moved code for </span><b>XLineColorItem</b> from the first to the latter. As it was pointed out last week, I'd have to do something with these lines:</div>
<div style><a href="http://opengrok.libreoffice.org/xref/core/svx/source/tbxctrls/tbcontrl.cxx#116">http://opengrok.libreoffice.org/xref/core/svx/source/tbxctrls/tbcontrl.cxx#116</a><br></div><div style><br></div><div style>
I think I might have to replace all occurrences of XLineColorItem with SvxColorItem?</div><div style><br></div><div style>I'd welcome some advice regarding whether these color types should be refactored, and if yes, how.</div>
<div style><br></div><div style="font-family:arial,sans-serif;font-size:13px">If anyone has any insights, please share them! Thank you!<font face="arial, sans-serif"><br></font></div><div style="font-family:arial,sans-serif;font-size:13px">
<font face="arial, sans-serif"><br></font></div><div style="font-family:arial,sans-serif;font-size:13px"><font face="arial, sans-serif">All the best,</font></div><div style="font-family:arial,sans-serif;font-size:13px"><font face="arial, sans-serif">Krisztian</font></div>
</div>