Making WordPress.org

Opened 2 years ago

Closed 3 months ago

#6125 closed defect (bug) (fixed)

Need to cursor pointer if element clickable

Reported by: sumitsingh's profile sumitsingh Owned by: coffee2code's profile coffee2code
Milestone: Priority: lowest
Component: Theme Directory Keywords: has-screenshots has-patch needs-testing
Cc:

Description

Hi ,

I have visited https://wordpress.org/themes/tags/grid-layout/ page. and there complete filter section clickable. So we need to add cursor pointer or somthing for user can see it si clicable complete section Or button for edit filter.

Quick video https://www.loom.com/share/8670335209c447d383ebf089d73bd6b3

Attachments (4)

#6125.patch (529 bytes) - added by kajalgohel 2 years ago.
#6125.2.patch (531 bytes) - added by kajalgohel 2 years ago.
Added css for all the tags(clickable),which selected for featured filter
#6125.3.patch (671 bytes) - added by kajalgohel 2 years ago.
Added patch for click event handler, which only works on a Edit link
6125.diff (573 bytes) - added by aezazshekh 2 years ago.
Added patch. ( Added cursor pointer )

Download all attachments as: .zip

Change History (9)

@kajalgohel
2 years ago

#1 @kajalgohel
2 years ago

  • Keywords has-patch added

@kajalgohel
2 years ago

Added css for all the tags(clickable),which selected for featured filter

#2 @dd32
2 years ago

  • Component changed from General to Theme Directory
  • Milestone Q1 deleted
  • Priority changed from normal to lowest

While a click event is handled, it doesn't seem relevant to change to a pointer here. The Edit link correctly has a pointer attached.

I'd probably sooner make the click handler appropriately only work on the link.

#3 @sumitsingh
2 years ago

Sound good @dd32

@kajalgohel
2 years ago

Added patch for click event handler, which only works on a Edit link

#4 @kajalgohel
2 years ago

  • Keywords needs-testing added

@aezazshekh
2 years ago

Added patch. ( Added cursor pointer )

#5 @coffee2code
3 months ago

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

In 13057:

Theme Directory: Remove click handler from filter tags.

Props kajalgohel, dd32, sumitsingh.
Fixes #6125.

Note: See TracTickets for help on using tickets.