WordPress.org

Make WordPress Core

Ticket #51754: 51754-1.diff

File 51754-1.diff, 3.6 KB (added by antpb, 4 months ago)

initial patch for design and feature discussion

  • src/wp-admin/async-upload.php

     
    4949        switch ( $_REQUEST['fetch'] ) {
    5050                case 3:
    5151                        $thumb_url = wp_get_attachment_image_src( $id, 'thumbnail', true );
    52                         if ( $thumb_url ) {
    53                                 echo '<img class="pinkynail" src="' . esc_url( $thumb_url[0] ) . '" alt="" />';
    54                         }
    5552                        if ( current_user_can( 'edit_post', $id ) ) {
    5653                                echo '<a class="edit-attachment" href="' . esc_url( get_edit_post_link( $id ) ) . '" target="_blank">' . _x( 'Edit', 'media item' ) . '</a>';
    57                         } else {
     54                        } else { 
    5855                                echo '<span class="edit-attachment">' . _x( 'Success', 'media item' ) . '</span>';
    5956                        }
     57                        if ( $thumb_url ) {
     58                                echo '<img class="pinkynail" src="' . esc_url( $thumb_url[0] ) . '" alt="" />';
     59                        }
    6060
    6161                        // Title shouldn't ever be empty, but use filename just in case.
    6262                        $file  = get_attached_file( $post->ID );
     63                        $fileurl  = wp_get_attachment_url( $post->ID );
    6364                        $title = $post->post_title ? $post->post_title : wp_basename( $file );
     65
     66                        // @todo make a class for this field.
     67                        ?>
     68                        <div class="edit-attachment">
     69                                <label for="attachment-details-copy-link" class="name"><?php _e( 'File URL:' ); ?></label>
     70                                <input type="text" class="attachment-details-copy-link copy-attachment-url" id="attachment-details-copy-link" value="<?php echo $fileurl; ?>" readonly />
     71                                <span class="copy-to-clipboard-container-uploader">
     72                                        <button type="button" class="button button-small copy-attachment-url-uploader" data-clipboard-target="#attachment-details-copy-link"><?php _e( 'Copy URL to clipboard' ); ?></button>
     73                                        <span class="success hidden" aria-hidden="true"><?php _e( 'Copied!' ); ?></span>
     74                                </span>
     75                        </div>
     76                        <?php                                           
    6477                        echo '<div class="filename new"><span class="title">' . esc_html( wp_html_excerpt( $title, 60, '&hellip;' ) ) . '</span></div>';
    6578                        break;
    6679                case 2:
  • src/wp-admin/css/media.css

     
    819819        margin-left: 8px;
    820820}
    821821
     822.copy-to-clipboard-container-uploader {
     823        display: inline;
     824        align-items: center;
     825        margin-top: 8px;
     826        margin-right: 4px;
     827        clear: both;
     828}
     829
     830.copy-to-clipboard-container-uploader .copy-attachment-url-uploader {
     831        white-space: normal;
     832        margin-top: 4px !important;
     833}
     834
     835.copy-to-clipboard-container-uploader .success {
     836        color: #008a20;
     837        margin-left: 8px;
     838}
     839
     840
     841
    822842/*------------------------------------------------------------------------------
    823843  14.2 - Image Editor
    824844------------------------------------------------------------------------------*/
  • src/wp-includes/script-loader.php

     
    899899                $scripts->add( "plupload-$handle", false, array( 'plupload' ), '2.1.1' );
    900900        }
    901901
    902         $scripts->add( 'plupload-handlers', "/wp-includes/js/plupload/handlers$suffix.js", array( 'plupload', 'jquery' ) );
     902        $scripts->add( 'plupload-handlers', "/wp-includes/js/plupload/handlers$suffix.js", array( 'clipboard', 'jquery', 'plupload', 'underscore', 'wp-a11y', 'wp-i18n' ) );
    903903        did_action( 'init' ) && $scripts->localize( 'plupload-handlers', 'pluploadL10n', $uploader_l10n );
    904904
    905905        $scripts->add( 'wp-plupload', "/wp-includes/js/plupload/wp-plupload$suffix.js", array( 'plupload', 'jquery', 'json2', 'media-models' ), false, 1 );