phinger-cursors icon indicating copy to clipboard operation
phinger-cursors copied to clipboard

Cursor size is inconsistent

Open D3SOX opened this issue 4 years ago • 6 comments

Hey,

I like the design of Phinger, but I have a problem with the cursor size.

For me, Breeze Light at size 24 has the perfect size. Phinger Light at size 24 is too small (especially the normal cursor) and size 32 is too big (especially the resize and text cursors).

I'm using KDE Plasma 5.23.90 on Arch btw using the AUR package version 1.1-3.


Breeze Light, Size 24:

Normal image Text image Resize image


Phinger Light, Size 24:

Normal image Text image Resize image


Phinger Light, Size 32

Normal image Text image Resize image


Off-topic

(is it named Phinger because of the word finger? xD)

D3SOX avatar Feb 01 '22 08:02 D3SOX

The resize cursor you are showing is not the phinger-cursors one. Which means the app you are resizing probably uses another name for the cursor, and it should be added. Which app is that?

Regarding the text cursor: is it too tall in your opinion? To me it seems like it's right at the sweet-spot where it just covers the line height: image

phisch avatar Feb 01 '22 09:02 phisch

In the resize example, I used Konsole. I compared it again and the text cursor is actually good. It's just a little smaller at 24 compared to Breeze at 24.

However, I still think the normal cursor is too small at 24. I wonder if it has the same size for you.

D3SOX avatar Feb 01 '22 10:02 D3SOX

image

Maybe unrelated to this, but I also noticed that the preview in Plasma's system settings doesn't show the pointer with the spinner.

D3SOX avatar Feb 01 '22 10:02 D3SOX

@D3SOX the spinner is related to #5 but in your screenshot the fleur cursor seems to be the wrong one as well. I was expecting a couple aliases to be missing, i didn't have too many people testing the theme during beta. No big deal.

For the 24x24 cursor, there isn't much space to make it bigger: image

phisch avatar Feb 01 '22 11:02 phisch

I meant the size of the default cursor without anything else. (Compare the relation from Breeze to Phinger at 24)

D3SOX avatar Feb 01 '22 12:02 D3SOX

I get it, but I need to leave that space empty in the default cursor, so when the "loading" cursor for example gets displayed, the cursor size doesn't suddenly switch. It should look like something was added to your cursor, and not like your cursor switched to a different sized one.

Just checked, and Breeze cursor comes in 32 and 64 image

So, there is no 24x24 sprite in there. It probably shows you the 32x32 version if you set it to 24.

phisch avatar Feb 01 '22 12:02 phisch

@D3SOX The new 2.0-beta.1 release has some changes to better utilize the available space. I'd love it if you can have a look and test to see if you are more happy with the cursor sizes now!

Here is an example (green dot means this is new, the other is old): image

Let me know what you think!

phisch avatar Apr 04 '24 22:04 phisch

The previews on plasma are also looking nice now: Screenshot_20240407_005646

phisch avatar Apr 06 '24 22:04 phisch