Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#5943 closed defect (bug) (fixed)

Buttons in admin area needs a cursor definition

Reported by: ionfish's profile ionfish Owned by: iammattthomas's profile iammattthomas
Milestone: 2.5 Priority: high
Severity: normal Version: 2.5
Component: Administration Keywords: has-patch needs-commit
Focuses: Cc:

Description

.button et al. need a cursor property; the cursor shows as an arrow when you mouseover, rather than a pointer. Issue appears in Safari 3 and Firefox 2 (both on a Mac).

Attachments (2)

cursorfix.diff (305 bytes) - added by ionfish 18 years ago.
5943.diff (323 bytes) - added by iammattthomas 18 years ago.
Re-apply pointer cursor for buttons.

Download all attachments as: .zip

Change History (15)

@ionfish
18 years ago

#1 @ionfish
18 years ago

  • Summary changed from Buttons in admin area are missing a cursor to Buttons in admin area need a cursor definition

Slight clarification of issue summary.

#2 @ionfish
18 years ago

  • Summary changed from Buttons in admin area need a cursor definition to Buttons in admin area needs a cursor definition

Spelling.

#3 @mdawaffe
18 years ago

  • Owner changed from anonymous to iammattthomas
  • Priority changed from normal to high

+1

If we're not going to have the buttons look "buttony", a pointer cursor makes the interaction seem way more natural.

With :hover color shift as the only visual feedback, I have no idea what's going to happen if I click the mouse. It makes me think some kind of javascript nonsense is about to happen :)

#4 @iammattthomas
18 years ago

  • Cc rboren added

I agree. Let's patch it up.

#5 @ryan
18 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [7040]) Add cursor pointer to buttons. Props ionfish. fixes #5943

#6 @mdawaffe
18 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Matt reverted this with [7184]. I disagree.

#7 @zamoose
18 years ago

I agree with mdawaffe's disagreement.

#8 @iammattthomas
18 years ago

Agree with mdawaffe -- there are places where buttons are inputs, and places where buttons are text links. If we don't specify the pointer cursor for all, they won't be consistent, and for no apparent reason to the user.

#9 @iammattthomas
18 years ago

  • Cc matt added; rboren removed

#10 @iammattthomas
18 years ago

  • Cc rboren added; matt removed

Happy Cog says the pointer cursor is the way to go. Can rboren just re-apply this patch?

#11 @ionfish
18 years ago

  • Keywords needs-commit added

Bit confused as to why this was reverted at all, but can it be reinstated please? Buttons lacking a proper cursor is a usability nightmare.

#12 @ryan
18 years ago

Create a fresh patch and I'll get it in.

@iammattthomas
18 years ago

Re-apply pointer cursor for buttons.

#13 @ryan
18 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [7215]) Add cursor pointer to buttons. Props ionfish and mt. fixes #5943

Note: See TracTickets for help on using tickets.