WordPress.org

Make WordPress Core

Ticket #5911: 5911-16.diff

File 5911-16.diff, 7.6 KB (added by andy, 10 years ago)

Improve upload_max_filesize error. Improve error display for long messages. New uploads now added to bottom, not top. Fix align class. Remove redundant header text.

  • wp-includes/js/swfupload/handlers.js

     
    1111function fileQueued(fileObj) {
    1212        // Get rid of unused form
    1313        jQuery('.media-blank').remove();
     14        // Collapse a single item
     15        if ( jQuery('.type-form #media-items>*').length == 1 && jQuery('#media-items .hidden').length > 0 ) {
     16                jQuery('.toggle').toggle();
     17                jQuery('.slidetoggle').slideUp(200).siblings().removeClass('hidden');
     18        }
    1419        // Create a progress bar containing the filename
    15         jQuery('#media-items').prepend('<div id="media-item-' + fileObj.id + '" class="media-item child-of-' + post_id + '"><span class="filename original">' + fileObj.name + '</span><div class="progress"><div class="bar"></div></div></div>');
     20        jQuery('#media-items').append('<div id="media-item-' + fileObj.id + '" class="media-item child-of-' + post_id + '"><div class="filename original">' + fileObj.name + '</div><div class="progress"><div class="bar"></div></div></div>');
    1621
    1722        // Disable the submit button
    1823        jQuery('#insert-media').attr('disabled', 'disabled');
     
    7681        if ( jQuery('.type-form #media-items>*').length == 1 ) {
    7782                jQuery('#media-items .slidetoggle').slideDown(500).parent().eq(0).children('.toggle').toggle();
    7883                jQuery('.type-form .slidetoggle').siblings().addClass('hidden');
    79         } else {
    80                 jQuery('.type-form .slidetoggle').siblings().removeClass('hidden');
    8184        }
    8285
    8386        // Only show Gallery button when there are at least two files.
     
    117120
    118121// file-specific message
    119122function wpFileError(fileObj, message) {
    120         jQuery('#media-item-' + fileObj.id + ' .filename').append('<span class="file-error">'+message+'</span> <button type="button" class="button dismiss">'+swfuploadL10n.dismiss+'</button>');
     123        jQuery('#media-item-' + fileObj.id + ' .filename').after('<div class="file-error"><button type="button" class="button dismiss">'+swfuploadL10n.dismiss+'</button>'+message+'</div>').siblings('.progress').remove();
    121124        jQuery('.dismiss').click(function(){jQuery(this).parents('.media-item').slideUp(200, function(){jQuery(this).remove();})});
    122125}
    123126
  • wp-includes/script-loader.php

     
    9797                // these error messages came from the sample swfupload js, they might need changing.
    9898                $this->localize( 'swfupload-handlers', 'swfuploadL10n', array(
    9999                                'queue_limit_exceeded' => __('You have attempted to queue too many files.'),
    100                                 'file_exceeds_size_limit' => __('This file is too big. See php.ini.'),
     100                                'file_exceeds_size_limit' => sprintf(__('This file is too big. Your php.ini upload_max_filesize is %s.'), ini_get('upload_max_filesize')),
    101101                                'zero_byte_file' => __('The file you selected is empty. Please select another file.'),
    102102                                'invalid_filetype' => __('The file you choose is not an allowed file type.'),
    103103                                'default_error' => __('An error occurred in the upload. Please try again later.'),
  • wp-admin/includes/media.php

     
    270270                $alt = attribute_escape($_POST['insertonly']['alt']);
    271271                if ( isset($_POST['insertonly']['align']) ) {
    272272                        $align = attribute_escape($_POST['insertonly']['align']);
    273                         $class = " class='align-$align'";
     273                        $class = " class='align$align'";
    274274                }
    275275                $html = "<img src='$src' alt='$alt'$class />";
    276276                return media_send_to_editor($html);
     
    602602
    603603        foreach ( $attachments as $id => $attachment )
    604604                if ( $item = get_media_item($id, isset($errors[$id]) ? $errors[$id] : null) )
    605                         $output .= "\n<div id='media-item-$id' class='media-item child-of-$attachment->post_parent preloaded'><div id='media-upload-error-$id'></div><span class='filename'></span><div class='progress'><div class='bar'></div></div>$item<div class='progress clickmask'></div>\n</div>";
     605                        $output .= "\n<div id='media-item-$id' class='media-item child-of-$attachment->post_parent preloaded'><div id='media-upload-error-$id'></div><div class='filename'></div><div class='progress'><div class='bar'></div></div>$item<div class='progress clickmask'></div>\n</div>";
    606606
    607607        return $output;
    608608}
     
    645645        $type
    646646        <a class='toggle describe-toggle-on' href='#'>$toggle_on</a>
    647647        <a class='toggle describe-toggle-off' href='#'>$toggle_off</a>
    648         <span class='filename new'>$filename</span>
     648        <div class='filename new'>$filename</div>
    649649        <table class='slidetoggle describe $class'><tbody>
    650650                <tr>
    651651                        <td class='A1B1' rowspan='4' colspan='2'><img class='thumbnail' src='$thumb_url' alt='' /></td>
     
    730730        return $item;
    731731}
    732732
    733 function media_upload_header($title = false) {
    734         if ( empty($title) )
    735                 $title = __('Choose a File');
     733function media_upload_header() {
    736734        ?>
    737735        <script type="text/javascript">post_id = <?php echo intval($_REQUEST['post_id']); ?>;</script>
    738736        <div id="media-upload-header">
    739         <h3><?php echo $title ?></h3>
    740737        <?php the_media_upload_tabs(); ?>
    741738        </div>
    742739        <?php
     
    856853}
    857854
    858855function media_upload_gallery_form($errors) {
    859         media_upload_header(__('Browse attached files'));
     856        media_upload_header();
    860857
    861858        $post_id = intval($_REQUEST['post_id']);
    862859
     
    894891function media_upload_library_form($errors) {
    895892        global $wpdb, $wp_query, $wp_locale, $type, $tab, $post_mime_types;
    896893
    897         media_upload_header(__('Browse Media Library'));
     894        media_upload_header();
    898895
    899896        $post_id = intval($_REQUEST['post_id']);
    900897
  • wp-admin/css/media.css

     
    11div#media-upload-header {
    22        background-color: #e4f2fd;
    33        margin: 0;
    4         padding: 1em 0.5em 0 0.5em;
     4        padding: 0 5px;
    55        font-weight: bold;
    66        position: relative;
    77        border-bottom: 1px solid #c6d9e9;
     8        height: 2.5em;
    89}
    910
    1011ul#sidemenu {
    1112        font-weight: normal;
    12         margin: 0 15px;
     13        margin: 0 5px;
    1314        position: absolute;
    14         right: 1em;
     15        left: 0px;
    1516        bottom: -1px;
    1617}
    1718
     
    2122        color: #f00;
    2223}
    2324
    24 .file-error {
    25         font-weight: bold;
    26         color: #f00;
    27         margin: 0 15px;
    28 }
    29 
    3025form {
    3126        margin: 1em;
    3227}
     
    5954        margin: 0 0 1em 0;
    6055        padding: 0;
    6156}
    62 /*
    63 .media-upload-form button.button-ok {
    64         float: right;
    65         background-color: #ebebeb;
    66         color: #1f4569;
    67         border: none;
    68         padding: 0.5em;
    69 }
    7057
    71 .media-upload-form a.button-cancel {
    72         float: right;
    73         background-color: #fff;
    74         color: #9a9a9a;
    75         font-size: 0.8em;
    76         text-decoration: underline;
    77         margin: 0.5em 1em;
    78 }
    79 */
    8058/* specific to the image upload form */
    8159.align .field label {
    8260        display: inline;
     
    128106        border-bottom: 1px solid #d0d0d0;
    129107        width: 623px;
    130108        position: relative;
     109        min-height: 36px;
    131110}
    132 span.filename {
    133         position: absolute;
    134         left: 46px;
    135         top: 0px;
     111.filename {
    136112        line-height: 36px;
     113        margin-left: 50px;
    137114        z-index: 2;
    138115}
     116button.dismiss {
     117        position: absolute;
     118        top: 5px;
     119        right: 5px;
     120        z-index: 4;
     121}
     122.file-error {
     123        margin: 0 0 5px 50px;
     124        font-weight: bold;
     125        color: #f00;
     126}
     127
    139128.progress {
     129        position: absolute;
     130        top: 0px;
     131        left: 0px;
    140132        width: 623px;
    141133        height: 36px;
    142134}
    143135.bar {
    144136        width: 0px;
    145         height: 36px;
     137        height: 100%;
    146138        background-color: #e8e8e8;
    147139        border-right: 3px solid #99d;
    148140}
     
    161153        border-top: 1px solid #d0d0d0;
    162154        padding: 5px;
    163155        width: 100%;
     156        clear: both;
    164157}
    165158.describe.startopen, .describe.startclosed {
    166159        display: none;
     
    201194        left: 0px;
    202195        cursor: pointer;
    203196        border: none;
    204         z-index: 10;
     197        z-index: 3;
     198        height: 36px;
    205199}
    206200tr.align td.field {
    207201        text-align: center;