Ledger-live-desktop: No mouse "pointer" cursor hovering over a content of a button

Created on 7 Jul 2018  路  7Comments  路  Source: LedgerHQ/ledger-live-desktop

Ledger Live Version and Operating System

  • Ledger Live 1.0.0-rc.2
  • Platform: macOS

Expected behavior

When hovering over a button, mouse cursor should be set as "pointer".

Actual behavior

When hovering over a button, mouse cursor is set as "pointer" only when inside button's padding area, but is not within button content (icon/text).

Steps to reproduce the behavior

Click on image to see the video:

Reproducing the current issue

design enhancement

Most helpful comment

After a talk with @khalilbenihoud, we've decided to remove the cursor pointer everywhere (except for links), at least until we rework a custom toolbar.

All 7 comments

Thanks for the issue.
It seems this is a current limitation of Electron https://github.com/electron/electron/issues/13104

it looks like https://electronjs.org/docs/api/frameless-window#custombuttonsonhover would fix it however you no longer see the native button which is not really good option to me

a

I see.

Might then be worth looking at alternative solutions, e.g. Atom app displays the native title bar:

Ledger Live (Desktop) - example of window frame on Atom app

But also on macOS apps usually don't have "pointer" cursors at all (click on image to see the video):

Atom app example

Or you can just keep an issue open until possibly Electron will resolve it...

to retake Atom's example, it does not look like so consistent.
I guess not using pointer at all for the header buttons could be a tmp solution. the fact it hover with a background color make it pretty clear it's an action.

a

IMO: If we choose to put cursor default on some buttons, we should do it for all buttons/interactive boxes (except for links like). It's a more native approach and I recommend.

If not, I would go for custombuttonsonhover which is pretty quick to implement.

cc @bscholving

After a talk with @khalilbenihoud, we've decided to remove the cursor pointer everywhere (except for links), at least until we rework a custom toolbar.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mautematico picture mautematico  路  4Comments

MaTthyU picture MaTthyU  路  4Comments

kravens picture kravens  路  3Comments

gre picture gre  路  3Comments

nicolasbigot picture nicolasbigot  路  3Comments