Changeset 7816

Timestamp:

11/01/2018 10:31:04 PM (6 years ago)

Author:

coffee2code

Message:

Developer: Add a column in the admin listing of posts for an indicator if the parsed post has an explanation.

Fixes #3900.

Location:

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer

Files:

• inc/explanations.php (modified) (3 diffs)
• scss/admin.scss (modified) (2 diffs)
• stylesheets/admin.css (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/explanations.php

@@ -49 +49 @@
         add_action( 'init',                    array( $this, 'register_post_type'     ), 0     );
         add_action( 'init',                    array( $this, 'remove_editor_support'  ), 100   );
+
+        // Admin.
         add_action( 'edit_form_after_title',   array( $this, 'post_to_expl_controls'  )        );
         add_action( 'edit_form_top',           array( $this, 'expl_to_post_controls'  )        );
@@ -54 +56 @@
         add_action( 'admin_menu',              array( $this, 'admin_menu'             )        );
         add_action( 'load-post-new.php',       array( $this, 'prevent_direct_creation')        );
+        // Add admin post listing column for explanations indicator.
+        add_filter( 'manage_posts_columns',       array( $this, 'add_post_column'     )        );
+        // Output checkmark in explanations column if post has an explanation.
+        add_action( 'manage_posts_custom_column', array( $this, 'handle_column_data'  ), 10, 2 );
 
         // Permissions.
@@ -543 +549 @@
     }
 
+    /**
+     * Adds a column in the admin listing of posts for parsed post types to
+     * indicate if they have an explanation.
+     *
+     * Inserted as first column after title column.
+     *
+     * @access public
+     *
+     * @param array $columns Associative array of post column ids and labels.
+     * @return array
+     */
+    public function add_post_column( $columns ) {
+        if ( ! empty( $_GET['post_type'] ) && DevHub\is_parsed_post_type( $_GET['post_type'] ) ) {
+            $index = array_search( 'title', array_keys( $columns ) );
+            $pos   = false === $index ? count( $columns ) : $index + 1;
+
+            $col_data = [ 'has_explanation' => sprintf(
+                '<span class="dashicons dashicons-info" title="%s"></span><span class="screen-reader-text">%s</span>',
+                esc_attr__( 'Has explanation?', 'wporg' ),
+                esc_html__( 'Explanation?', 'wporg' )
+            ) ];
+            $columns  = array_merge( array_slice( $columns, 0, $pos ), $col_data, array_slice( $columns, $pos ) );
+        }
+
+        return $columns;
+    }
+
+    /**
+     * Outputs an indicator for the explanations column if post has an explanation.
+     *
+     * @access public
+     *
+     * @param string $column_name The name of the column.
+     * @param int    $post_id     The ID of the post.
+     */
+    public function handle_column_data( $column_name, $post_id ) {
+        if ( 'has_explanation' === $column_name ) {
+            if ( $explanation = DevHub\get_explanation( $post_id ) ) {
+                printf(
+                    '<a href="%s">%s%s</a>',
+                    get_edit_post_link( $explanation ),
+                    '<span class="dashicons dashicons-info" aria-hidden="true"></span>',
+                    '<span class="screen-reader-text">' . __( 'Post has an explanation.', 'wporg' ) . '</span>'
+                );
+            }
+        }
+    }
+
 }
 

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/scss/admin.scss

@@ -25 +25 @@
 
 /* Explanations */
+
+.fixed {
+    .column-has_explanation {
+        width: 2em;
+
+        .dashicons {
+            width: 30px;
+        }
+    }
+
+    tbody .column-has_explanation a {
+        color: #72777c;
+        display: inline-block;
+        margin-top: 4px;
+
+        &:focus, &:hover {
+            color: #0073aa;
+        }
+
+        .dashicons {
+            font-size: 26px;
+        }
+
+        .screen-reader-text {
+            @media screen and (max-width: 782px) {
+                position: static;
+                -webkit-clip-path: none;
+                clip-path: none;
+                width: auto;
+                height: auto;
+                margin: 0;
+            }
+        }
+
+        & [aria-hidden="true"] {
+            @media screen and (max-width: 782px) {
+                display: none;
+            }
+        }
+    }
+}
 
 .post-type-wporg_explanations .page-title-action {
@@ -81 +122 @@
     width: 100%;
 }
+

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/stylesheets/admin.css

@@ -23 +23 @@
 
 /* Explanations */
+.fixed .column-has_explanation {
+  width: 2em;
+}
+
+.fixed .column-has_explanation .dashicons {
+  width: 30px;
+}
+
+.fixed tbody .column-has_explanation a {
+  color: #72777c;
+  display: inline-block;
+  margin-top: 4px;
+}
+
+.fixed tbody .column-has_explanation a:focus, .fixed tbody .column-has_explanation a:hover {
+  color: #0073aa;
+}
+
+.fixed tbody .column-has_explanation a .dashicons {
+  font-size: 26px;
+}
+
+@media screen and (max-width: 782px) {
+  .fixed tbody .column-has_explanation a .screen-reader-text {
+    position: static;
+    -webkit-clip-path: none;
+    clip-path: none;
+    width: auto;
+    height: auto;
+    margin: 0;
+  }
+}
+
+@media screen and (max-width: 782px) {
+  .fixed tbody .column-has_explanation a [aria-hidden="true"] {
+    display: none;
+  }
+}
+
 .post-type-wporg_explanations .page-title-action {
   display: none;