Changeset 7086

Show
Ignore:
Timestamp:
02/28/08 09:11:11 (11 months ago)
Author:
ryan
Message:

Widget admin layout fixes from mdawaffe. fixes #5997

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/wp-admin/css/widgets-rtl.css

    r6754 r7086  
    3535} 
    3636 
     37li.widget-list-control-item h4.widget-title a, 
     38#dragHelper li.widget-list-control-item h4.widget-title a, 
     39#draghelper li.widget-list-control-item h4.widget-title a:visited { 
     40    right: auto; 
     41    left: 2em; 
     42} 
     43 
    3744li.widget-list-control-item div.widget-control { 
    3845    padding: 0 0 0 10px; 

  • trunk/wp-admin/css/widgets.css

    r6754 r7086  
    1515    clear: right; 
    1616    width: 300px; 
     17    position: relative; 
    1718} 
    1819 
    …… 
    9697 
    9798li.widget-list-control-item h4, #dragHelper li.widget-list-control-item h4 { 
    98     position: relative; 
    9999    margin: 0; 
    100100    background-color: #2683ae; 
    …… 
    114114} 
    115115 
    116 li.widget-list-control-item h4.widget-title a, li.widget-list-control-item h4.widget-title a:visited { 
     116li.widget-list-control-item h4.widget-title a, 
     117#dragHelper li.widget-list-control-item h4.widget-title a, 
     118#draghelper li.widget-list-control-item h4.widget-title a:visited { 
    117119    color: #fff; 
     120    right: 2em; 
    118121} 
    119122 

  • trunk/wp-admin/includes/widgets.php

    r7080 r7086  
    6464            } else { 
    6565                $action = 'edit'; 
    66                 $edit_url = add_query_arg( array( 
     66                $edit_url = clean_url( add_query_arg( array( 
    6767                    'sidebar' => $sidebar, 
    6868                    'edit' => $widget['id'], 
    6969                    'key' => array_search( $widget['id'], $sidebars_widgets[$sidebar] ), 
    70                 ) ); 
    71                 $widget_control_template = '<textarea>' . htmlspecialchars( $widget_control_template ) . '</textarea>'; 
     70                ) ) ); 
     71                $widget_control_template = '<li><textarea rows="1" cols="1">' . htmlspecialchars( $widget_control_template ) . '</textarea></li>'; 
    7272            } 
    7373 

  • trunk/wp-admin/js/widgets.js

    r6679 r7086  
    77    var toggleWidget = function( li ) { 
    88        var width = li.find('input.widget-width').val(); 
    9         return li.children('div.widget-control').each( function() { 
     9 
     10        // it seems IE chokes on these animations because of the positioning/floating 
     11        var widgetAnim = $.browser.msie ? function() { 
    1012            var t = $(this); 
    1113            if ( t.is(':visible') ) { 
    12                 t.animate( { height: 'hide' } ); 
     14                li.css( 'marginLeft', 0 ); 
     15                t.siblings('h4').children('a').text( widgetsL10n.edit ); 
     16            } else { 
     17                if ( width > 250 ) 
     18                    li.css( 'marginLeft', ( width - 250 ) * -1 ); 
     19                t.siblings('h4').children('a').text( widgetsL10n.cancel ); 
     20            } 
     21            t.toggle(); 
     22        } : function() { 
     23            var t = $(this); 
     24 
     25            if ( t.is(':visible') ) { 
    1326                if ( width > 250 ) 
    1427                    li.animate( { marginLeft: 0 } ); 
    1528                t.siblings('h4').children('a').text( widgetsL10n.edit ); 
    1629            } else { 
    17                 t.animate( { height: 'show' } ); 
    1830                if ( width > 250 ) 
    1931                    li.animate( { marginLeft: ( width - 250 ) * -1 } ); 
    2032                t.siblings('h4').children('a').text( widgetsL10n.cancel ); 
    2133            } 
    22         } ).end(); 
     34            t.animate( { height: 'toggle' } ); 
     35        }; 
     36 
     37        return li.children('div.widget-control').each( widgetAnim ).end(); 
    2338    }; 
    2439 

  • trunk/wp-includes/js/wp-lists.js

    r7082 r7086  
    99        var pp = s.split('&'); 
    1010        for ( var i in pp ) { 
     11            if ( $.isFunction(pp.hasOwnProperty) && !pp.hasOwnProperty(i) ) { continue; } 
    1112            var p = pp[i].split('='); 
    1213            r[p[0]] = p[1];