Make WordPress Core

Opened 5 years ago

Last modified 5 years ago

#45079 new defect (bug)

Clear inactive widgets button issue in IE Edge

Reported by: mukesh27's profile mukesh27 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Widgets Keywords: has-patch 2nd-opinion
Focuses: administration Cc:

Description

In Internet Explorer Edge system allow to click even if there is no widgets in Inactive Widgets panel. check video https://www.youtube.com/watch?v=GGseMV0FTeo

Attachments (1)

45079.diff (560 bytes) - added by xkon 5 years ago.
Checks if "Clear inactive widgets" input is disabled - IE fix

Download all attachments as: .zip

Change History (6)

#1 @knutsp
5 years ago

Edge != Internet Explorer (IE)

Edge, IE or both affected?

Last edited 5 years ago by knutsp (previous) (diff)

#2 follow-up: @mukesh27
5 years ago

@knutsp check attached video for more information.

#3 in reply to: ↑ 2 ; follow-up: @knutsp
5 years ago

Replying to mukesh27:

@knutsp check attached video for more information.

I did before commenting. It describes the problem, briefly, but can't see an answer to my question.

#4 in reply to: ↑ 3 @mukesh27
5 years ago

In both Edge and IE.

Replying to knutsp:

Replying to mukesh27:

@knutsp check attached video for more information.

I did before commenting. It describes the problem, briefly, but can't see an answer to my question.

@xkon
5 years ago

Checks if "Clear inactive widgets" input is disabled - IE fix

#5 @xkon
5 years ago

  • Keywords has-patch 2nd-opinion added; needs-patch removed

This was reproducable on IE ( 11 in my case ) but not on Edge. The wpWidgets.removeInactiveWidgets() was running in IE even though the input had a disabled state.

In 45079.diff I've added an extra check to see if the input is actually disabled before running the wpWidgets.removeInactiveWidgets().

Note: See TracTickets for help on using tickets.