Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#39426 closed defect (bug) (duplicate)

Adding new widget in Customizer shows Delete control instead of Remove

Reported by: monikarao Owned by:
Priority: normal Milestone:
Component: Customize Version: 4.7
Severity: normal Keywords: has-patch reporter-feedback needs-testing
Cc: Focuses: ui

Description

I noticed that in Customizer, existing widgets are showing red colored "Remove" control but when we add new widget by clicking "Add a Widget", it shows "Delete" control instead of "Remove".

Please refer attached screenshots "customizer-existing-widget.png" and "customizer-new-widget.png"

Attachments (4)

customizer-existing-widget.png (27.0 KB ) - added by monikarao 10 years ago.
Customizer - "Remove" control in existing widget
customizer-new-widget.png (15.4 KB ) - added by monikarao 10 years ago.
Customizer - "Delete" control in newly added widget
39426.diff (581 bytes ) - added by bhargavbhandari90 10 years ago.
Changed text from "Delete" to "Remove"
39426.2.diff (762 bytes ) - added by jainnidhi 10 years ago.

Download all attachments as: .zip

Change History (9)

@monikarao
10 years ago

Customizer - "Remove" control in existing widget

@monikarao
10 years ago

Customizer - "Delete" control in newly added widget

@bhargavbhandari90
10 years ago

Changed text from "Delete" to "Remove"

#1 in reply to: ↑ description @bhargavbhandari90
10 years ago

  • Keywords has-patch reporter-feedback added

Replying to monikarao:

I noticed that in Customizer, existing widgets are showing red colored "Remove" control but when we add new widget by clicking "Add a Widget", it shows "Delete" control instead of "Remove".

Please refer attached screenshots "customizer-existing-widget.png" and "customizer-new-widget.png"

Yes @monikarao it should be same text. Added a patch for this. This should be fine now.

#2 follow-ups: @ibachal
10 years ago

@bhargavbhandari90 As per your patch, this will also impact the controls in backend widgets Appearance > Widgets.

@jainnidhi
10 years ago

#3 in reply to: ↑ 2 @jainnidhi
10 years ago

  • Keywords needs-testing added

So, we need to replace "Remove" with "Delete". Patch "39426.2.diff" is attached. It will change the control "Remove" to "Delete" only in Customizer widgets.

Replying to ibachal:

@bhargavbhandari90 As per your patch, this will also impact the controls in backend widgets Appearance > Widgets.

Last edited 10 years ago by jainnidhi (previous) (diff)

#4 in reply to: ↑ 2 @bhargavbhandari90
10 years ago

Replying to ibachal:

@bhargavbhandari90 As per your patch, this will also impact the controls in backend widgets Appearance > Widgets.

I did it for making same word everywhere.

#5 @westonruter
10 years ago

  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

Thanks for the report and conversation. However, the Delete vs Remove distinction is intentional. See #27575.

Note: See TracTickets for help on using tickets.