Opened 5 years ago
Closed 5 years ago
#5943 closed defect (bug) (fixed)
Buttons in admin area needs a cursor definition
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 2.5 |
| Component: | Administration | Version: | 2.5 |
| Severity: | normal | Keywords: | has-patch needs-commit |
| Cc: | rboren |
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)
Change History (15)
- Summary changed from Buttons in admin area are missing a cursor to Buttons in admin area need a cursor definition
- Summary changed from Buttons in admin area need a cursor definition to Buttons in admin area needs a cursor definition
Spelling.
- 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 :)
- Resolution set to fixed
- Status changed from new to closed
- Resolution fixed deleted
- Status changed from closed to reopened
Matt reverted this with [7184]. I disagree.
comment:8
iammattthomas — 5 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.
comment:9
iammattthomas — 5 years ago
- Cc matt added; rboren removed
comment:10
iammattthomas — 5 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?
comment:11
ionfish — 5 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.
comment:12
ryan — 5 years ago
Create a fresh patch and I'll get it in.
comment:13
ryan — 5 years ago
- Resolution set to fixed
- Status changed from reopened to closed

Slight clarification of issue summary.