Fix looks good. Approved.
On 10-Jun-20 0:09, Sergey Bylokhov wrote:
On 6/8/20 10:55 am, Alexander Zuev wrote:
Hi Sergey,
I see that in show() you have removed specific workaround for
Solaris.
I guess this is justified but shouldn't we put some comments that
will prevent
backporting this fix to the version where Solaris support is still
relevant?
That code was not removed, it is became the main code path. After the
fix, we
always hide the component so it will not jump around during
initialization.