fix: change min cursor padding to 0

[drendog]

Describe your PR, what does it fix/add?

Since ed411f5 a padding has been added to the cursor, which means that some application that expect a hit in the window border bounds, will never detects the hit there.
For example when using firefox at full screen mode, that expect a hit on top of the windows to show the hidden tabs, it will never detects the hit.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

ed411f5 #5902

Is it ready for merging, or does it need work?

Ready

[Agent00Ming]

but my scroll bar 😭 #5902 (comment) (JK, but this could be turned into a per-side config maybe)

[Agent00Ming]

Firefox fullscreen functionality is reduced by the 1 pixel minimum. This does indeed fix it.

[vaxerski]

wouldnt it be better to add a setting to have the padding only on the bottom right?

[morr0ne]

wouldnt it be better to add a setting to have the padding only on the bottom right?

Maybe I am not understanding this right, but what is the purpose of the padding? It mainly seems to break stuff.

[Agent00Ming]

wouldnt it be better to add a setting to have the padding only on the bottom right?

Maybe I am not understanding this right, but what is the purpose of the padding? It mainly seems to break stuff.

#5902 (comment) and #5902 (comment) should explain the reason behind it.

[morr0ne]

Okay after reading through the pr I understand the purpose of the padding. However it feels like it should default to 0 and then users affected can add the 1px padding themselves. If this gets released as is most desktop users will end up with a broken config by default without even knowing why.

+1 for this pr

[vaxerski]

I still think this is the wrong solution and instead #6027 (comment) should be done

[morr0ne]

I still think this is the wrong solution and instead #6027 (comment) should be done

That could be a solution. Regardless this should be considered a blocker for 0.41.0 until resolved imo