<div dir="ltr"><div dir="ltr"><div dir="ltr"><div></div><div>I need a code reviewer for a patch I submitted to gerrit for bug <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=104169">tdf#104169</a>.<br></div><div>Patch : <a href="https://gerrit.libreoffice.org/c/core/+/114679">https://gerrit.libreoffice.org/c/core/+/114679</a></div><div>Bug : <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=104169">https://bugs.documentfoundation.org/show_bug.cgi?id=104169</a></div><div><br></div><div>The bug is a difficultyInteresting EasyHack and involves Calc and mouse cursors. It is actually an enhancement to calc and aims at introducing a new cross cursor to replace the current Arrow pointer while hovering over the spreadsheet in calc.</div><div><br></div><div>This patch is in continuation to an unfinished patch submitted by Mesut Cifci an year ago (<a href="https://gerrit.libreoffice.org/c/core/+/89744">https://gerrit.libreoffice.org/c/core/+/89744</a>). At the time Mesut was working on this bug, a cross cursor was already present in calc but it was the thin, black-coloured cross used for auto-fill. Mesut added another thicker, white cross to use in this bug, which goes by the name FatCross in code.</div><div><br></div><div>I  adjusted cursor hotspot and fixed some cursor calls on mouse clicks in code and have (apparently not) completed the enhancement. I successfully tested the new cursor on my Linux system. It looks like this: <a href="https://bugs.documentfoundation.org/attachment.cgi?id=171459">https://bugs.documentfoundation.org/attachment.cgi?id=171459</a>. But Heiko soon reported an issue with the appearance of the cursor on his configuration (another flavour of Linux): <a href="https://bugs.documentfoundation.org/attachment.cgi?id=171447">https://bugs.documentfoundation.org/attachment.cgi?id=171447</a>. I have, so far, failed to replicate the problem on my system. (More details on <a href="https://gerrit.libreoffice.org/c/core/+/114679">gerrit</a>)</div><div><br></div><div>So I need someone to review my code (and perhaps help a bit if needed).</div><div>An opinion or suggestion on the cursor's appearance would also be appreciated.</div><div><br></div><div>Thanks</div><div>Sarabjot Singh</div></div></div></div>