web-maker icon indicating copy to clipboard operation
web-maker copied to clipboard
verified publisher icon chinchang

Added cursor pointer to buttons and links in the footer

Open PaulvdDool opened this issue 6 years ago • 2 comments

Footer buttons and links had the default cursor on hover.

See link below for GIF with changes: https://jmp.sh/yb1a0rR

PaulvdDool avatar Apr 05 '19 13:04 PaulvdDool

Thanks for contributing @PaulvdDool. I reviewed your changes and have a few points:

  1. The cursor change on <a> isn't required. <a> already has pointer cursor.
  2. I am curious about the reason for changing the cursor on <button>. Is it something you felt the need for or just a convention you follow?

chinchang avatar Apr 13 '19 17:04 chinchang

This question is a really complex one. Normally button should not have pointer, as W3C explain that the pointer should show a link. From a user perspective, we started to get used to have the pointer to anything that could get clickable. But it's not an easy choice and I just wanted to be sure that both of you had read this article before merging the changes:

https://medium.com/simple-human/buttons-shouldnt-have-a-hand-cursor-b11e99ca374b

julientaq avatar Jun 03 '19 08:06 julientaq

Thanks for the contribution @PaulvdDool 🙇 This took a while to get merged 😅 PS: merged it manually since it was out of date

chinchang avatar Oct 13 '23 10:10 chinchang