search

silverstripe / silverstripe-ckan-registry

BSD 3-Clause "New" or "Revised" License
6 stars 3 forks source link

Remove hover on GridField rows #168

Open brynwhyman opened 5 years ago

brynwhyman commented 5 years ago

Remove the mouse-over hover on the rows

robbieaverill commented 5 years ago

Are you referring to the columns and filters GridFields for CKAN pages in the CMS?

If so, do you want to raise this as a core change, since we're using a core design pattern here?

ScopeyNZ commented 5 years ago

Yeah this has a separate "bigger" issue against GridField extensions as it's technically some weird UX when you use GridFieldEditableRows. There's still a recommendation that we implement a work around though.

robbieaverill commented 5 years ago

I think we should remove cursor: pointer but leave the hover background change, personally

sachajudd commented 5 years ago

Related: https://github.com/symbiote/silverstripe-gridfieldextensions/issues/279