Merge pull request #610 from ProgressPlanner/filip/v18/todo-styling

Todo list tweaks

Author: aristath

assets/css/page-widgets/todo.css

@@ -210,6 +210,7 @@
 			font-size: var(--prpl-font-size-small);
 
 			&:hover {
+				text-decoration: underline;
 
 				svg path {
 					fill: var(--prpl-color-accent-red);
@@ -342,7 +343,12 @@
 			padding: 0;
 			margin: 0;
 			cursor: pointer;
-			text-decoration: underline;
+
+			&:hover,
+			&:focus,
+			&:active {
+				text-decoration: underline;
+			}
 		}
 	}
 }

assets/js/suggested-task.js

@@ -545,6 +545,7 @@ prplSuggestedTask = {
 		if ( event.key === 'Enter' ) {
 			event.preventDefault();
 			event.stopPropagation();
+			event.target.blur();
 			return false;
 		}
 	},

classes/suggested-tasks/providers/class-user.php

@@ -89,7 +89,7 @@ public function get_task_details( $task_data = [] ) {
 	public function add_task_actions( $data = [], $actions = [] ) {
 		$actions[] = [
 			'priority' => 10,
-			'html'     => '<a class="prpl-tooltip-action-text" href="#" target="_self" onclick="this.closest(\'li.prpl-suggested-task\').querySelector(\'.prpl-task-title span\').focus();">' . \esc_html__( 'Edit', 'progress-planner' ) . '</a>',
+			'html'     => '<a class="prpl-tooltip-action-text" href="#" target="_self" onclick="event.preventDefault();this.closest(\'li.prpl-suggested-task\').querySelector(\'.prpl-task-title span\').focus();">' . \esc_html__( 'Edit', 'progress-planner' ) . '</a>',
 		];
 
 		return $actions;