Hi Julien,

Oh sorry, I didn't realize this pattern is:

SetWidth(GetHeight())
SetHeight(GetHeight())

I thought it's:

SetWidth(GetHeight())
SetHeight(GetWidth())

With that in mind, the current code is just OK, the width of the
checkbox portion is the same as its height, since it's a square (and
that size depends on the font size).

Perhaps add a comment explaining this to make it easier for the next
person to understand. :-)

Regards,

Miklos

Attachment: signature.asc
Description: Digital signature

_______________________________________________
LibreOffice mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to