WordPress.org

Make WordPress Core

Opened 22 months ago

Closed 22 months ago

Last modified 22 months ago

#18582 closed defect (bug) (fixed)

Remove unneeded siblings('.widget-holder') from widgets.js

Reported by: aaroncampbell Owned by: azaozz
Priority: normal Milestone: 3.3
Component: Widgets Version: 2.9
Severity: normal Keywords: has-patch
Cc:

Description

Currently we do this:
$(this).siblings('.widget-holder').parent().toggleClass('closed');
However, since the parent() of the current element is the same as the parent of it's siblings I don't think we need to move to a sibling at all.

Attachments (1)

18582.diff​ (509 bytes) - added by aaroncampbell 22 months ago.

Download all attachments as: .zip

Change History (3)

aaroncampbell — 22 months ago

comment:1 azaozz — 22 months ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In [18634]:

Remove unneeded .siblings() from widgets.js, props aaroncampbell, fixes #18582

comment:2 ocean90 — 22 months ago

  • Milestone changed from Awaiting Review to 3.3
Note: See TracTickets for help on using tickets.