Make WordPress Core

Opened 2 years ago

Last modified 6 months ago

#55501 new defect (bug)

Multiple loader issue on admin general setting page.

Reported by: bhaveshkhadodara's profile bhaveshkhadodara Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.9.2
Component: Administration Keywords: has-patch changes-requested
Focuses: administration, multisite Cc:

Description

When you select a new language from the general settings page and click submit multiple times, multiple loader images are appended.

Step 1: Open settings -> general page
Step 2: Select a new language from the site language option.
Step 3: click the "Save Changes" button multiple times.

Attachments (4)

Loader Issue on setting page.png (164.0 KB) - added by bhaveshkhadodara 2 years ago.
Loader Issue on setting page.2.png (164.0 KB) - added by bhaveshkhadodara 2 years ago.
55501.patch (1.0 KB) - added by jariwalasagar 2 years ago.
Applied patch for this ticket.
55501.2.patch (1.3 KB) - added by dilipbheda 2 years ago.
Updated patch to avoid selecting a second time same element.

Download all attachments as: .zip

Change History (6)

@jariwalasagar
2 years ago

Applied patch for this ticket.

#1 @jariwalasagar
2 years ago

  • Keywords has-patch added; needs-patch removed

@dilipbheda
2 years ago

Updated patch to avoid selecting a second time same element.

#2 @sabernhardt
6 months ago

  • Component changed from General to Administration
  • Keywords changes-requested added

If the submit button (input) should be disabled, using .prop( 'disabled', true ) would be better than the .attr() method.
https://api.jquery.com/prop/#entry-longdesc-1

Note: See TracTickets for help on using tickets.