From 03fc7129109aeb3b98cace2ce9f41b5f44118c17 Mon Sep 17 00:00:00 2001 From: Alec Rust Date: Fri, 5 Aug 2022 18:16:05 +0100 Subject: [PATCH] Fix "pointer" cursor on status indicator Problem: User gets "pointer" cursor when hovering over the status indicator, even when it's green and does nothing Solution: Only apply this "pointer" cursor when in "error" state i.e. when hovering over the element does something --- build/scss/admin.scss | 2 +- src/MailAdminTable.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/build/scss/admin.scss b/build/scss/admin.scss index d57eb70..c75e45f 100644 --- a/build/scss/admin.scss +++ b/build/scss/admin.scss @@ -53,10 +53,10 @@ width: 12px; border-radius: 100%; margin: 5px auto 0 auto; - cursor: pointer; background-color: #35bc35; &.-error { + cursor: pointer; background-color: #fd4f4f; } } diff --git a/src/MailAdminTable.php b/src/MailAdminTable.php index ffdf41f..e7aef97 100644 --- a/src/MailAdminTable.php +++ b/src/MailAdminTable.php @@ -156,7 +156,7 @@ function prepare_items() $this->process_bulk_action(); $overrideParams = array_intersect_key($_REQUEST, Logs::$whitelistedParams); - + $this->items = Logs::get(array_merge([ 'paged' => $this->get_pagenum(), 'post_status' => isset($_GET['post_status']) ? $_GET['post_status'] : 'any',