WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#19594 closed defect (bug) (fixed)

Admin bar search button should indicate it's clickable

Reported by: jaquers Owned by: nacin
Priority: normal Milestone: 3.4
Component: Toolbar Version:
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

When you hover over the search button in the admin bar, the cursor should indicate that it's clickable.

Attachments (3)

admin-bar.dev.css​ (12.7 KB ) - added by jaquers 15 years ago.
Added cursor: pointer to a couple rules.
admin-bar.dev.patch​ (497 bytes ) - added by jaquers 15 years ago.
admin-bar.dev.css.patch​ (897 bytes ) - added by jaquers 15 years ago.
Updated patch to address focus state cursor appearance.

Download all attachments as: .zip

Change History (12)

@jaquers
15 years ago

Added cursor: pointer to a couple rules.

@jaquers
15 years ago

#1 @jaquers
15 years ago

Sorry for my noobness about filing and creating patches.. Hopefully you can overlook for the small change that this is :)

#2 @SergeyBiryukov
15 years ago

According to ticket:18197:146, current behaviour is proper.

#3 @SergeyBiryukov
15 years ago

  • Keywords ui-feedback added

#4 @nacin
15 years ago

  • Component General → Toolbar
  • Milestone Awaiting Review → 3.4

Yeah, but now that it's an icon, rather than a short text box that grows, it should get a hand.

#5 @SergeyBiryukov
15 years ago

With the current patch, the pointer cursor is displayed even when the text box is expanded.

@jaquers
15 years ago

Updated patch to address focus state cursor appearance.

#6 @jaquers
15 years ago

  • Keywords needs-testing added

#7 @SergeyBiryukov
15 years ago

admin-bar.dev.css.patch​ looks good. Tested in Firefox 9, Chrome 16, Opera 11.60, IE 8.

#8 @ocean90
15 years ago

  • Keywords commit added; ui-feedback needs-testing removed

Works.

#9 @nacin
15 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In [19703]:

Show cursor for the search icon. props jaquers. fixes #19594.

Note: See TracTickets for help on using tickets.