From b4f5d818b702590386299e8421be76f598d0c6d7 Mon Sep 17 00:00:00 2001 From: Huzaifa Al Mesbah Date: Thu, 29 Jan 2026 10:49:09 +0600 Subject: [PATCH 1/2] Widgets: Remove unused variables in widget toggle handler --- src/js/_enqueues/admin/widgets.js | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/src/js/_enqueues/admin/widgets.js b/src/js/_enqueues/admin/widgets.js index e8fc42507d382..8630f8d8a4253 100644 --- a/src/js/_enqueues/admin/widgets.js +++ b/src/js/_enqueues/admin/widgets.js @@ -113,15 +113,13 @@ window.wpWidgets = { }); $(document.body).on('click.widgets-toggle', function(e) { - var target = $(e.target), css = {}, - widget, inside, targetWidth, widgetWidth, margin, saveButton, widgetId, + var target = $(e.target), + widget, inside, saveButton, widgetId, toggleBtn = target.closest( '.widget' ).find( '.widget-top button.widget-action' ); if ( target.parents('.widget-top').length && ! target.parents('#available-widgets').length ) { widget = target.closest('div.widget'); inside = widget.children('.widget-inside'); - targetWidth = parseInt( widget.find('input.widget-width').val(), 10 ); - widgetWidth = widget.parent().width(); widgetId = inside.find( '.widget-id' ).val(); // Save button is initially disabled, but is enabled when a field is changed. @@ -137,16 +135,6 @@ window.wpWidgets = { } if ( inside.is(':hidden') ) { - if ( targetWidth > 250 && ( targetWidth + 30 > widgetWidth ) && widget.closest('div.widgets-sortables').length ) { - if ( widget.closest('div.widget-liquid-right').length ) { - margin = isRTL ? 'margin-right' : 'margin-left'; - } else { - margin = isRTL ? 'margin-left' : 'margin-right'; - } - - css[ margin ] = widgetWidth - ( targetWidth + 30 ) + 'px'; - widget.css( css ); - } /* * Don't change the order of attributes changes and animation: * it's important for screen readers, see ticket #31476. From 2e338c2291126bbb33db8925cc2798fae543e3ac Mon Sep 17 00:00:00 2001 From: Huzaifa Al Mesbah Date: Thu, 29 Jan 2026 23:27:55 +0600 Subject: [PATCH 2/2] Admin Widgets: Remove unused isRTL variable. --- src/js/_enqueues/admin/widgets.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/js/_enqueues/admin/widgets.js b/src/js/_enqueues/admin/widgets.js index 8630f8d8a4253..745f3fde7bd7f 100644 --- a/src/js/_enqueues/admin/widgets.js +++ b/src/js/_enqueues/admin/widgets.js @@ -2,7 +2,7 @@ * @output wp-admin/js/widgets.js */ -/* global ajaxurl, isRtl, wpWidgets */ +/* global ajaxurl, wpWidgets */ (function($) { var $document = $( document ); @@ -27,8 +27,7 @@ window.wpWidgets = { self = this, chooser = $('.widgets-chooser'), selectSidebar = chooser.find('.widgets-chooser-sidebars'), - sidebars = $('div.widgets-sortables'), - isRTL = !! ( 'undefined' !== typeof isRtl && isRtl ); + sidebars = $('div.widgets-sortables'); // Handle the widgets containers in the right column. $( '#widgets-right .sidebar-name' )