Make WordPress Core

Opened 4 years ago

Closed 11 months ago

#40260 closed enhancement (fixed)

Consider to remove some left overs from the old jQuery.suggest

Reported by: afercia Owned by: SergeyBiryukov
Milestone: 5.6 Priority: normal
Severity: normal Version: 4.7
Component: Administration Keywords: deprecated-css has-patch needs-testing
Focuses: Cc:


Noticed while investigating on #40242. Core doesn't use jquery.suggest since [38797], however there are some left overs here and there that reference the ac_results class used for the jquery.suggest results.

The JavaScript part can be safely removed. The CSS part might still be used by some very old plugins so maybe better to keep it and add some inline comment.

See quick discussion on Slack:

Aside: Maybe one more reason to add a deprecated.css somewhere, as this idea has been around since a while. See for example #21456, #26396, and also https://make.wordpress.org/core/2014/12/19/core-css-roadmap/

Attachments (4)

40260.diff (1.1 KB) - added by afercia 4 years ago.
40260.1.diff (1.0 KB) - added by Hareesh Pillai 22 months ago.
Patch refreshed
40260.2.diff (1.0 KB) - added by Hareesh Pillai 12 months ago.
40260.3.diff (1.0 KB) - added by garrett-eclipse 12 months ago.
Clear up comment verbiage. New verbiage "Back-compat styles from deprecated jQuery.suggest, see ticket #40260."

Download all attachments as: .zip

Change History (11)

#1 @afercia
4 years ago

See also #27555 where the decision was to keep the CSS in core for backwards compatibility.

4 years ago

#2 @afercia
4 years ago

  • Keywords has-patch added

@Hareesh Pillai
22 months ago

Patch refreshed

This ticket was mentioned in Slack in #core by hareesh-pillai. View the logs.

12 months ago

#4 @SergeyBiryukov
12 months ago

  • Milestone changed from Awaiting Review to 5.6
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

@Hareesh Pillai
12 months ago


12 months ago

Clear up comment verbiage. New verbiage "Back-compat styles from deprecated jQuery.suggest, see ticket #40260."

#5 @garrett-eclipse
12 months ago

Quick refresh in 40260.3.diff to correct the comment language.

P.S. I'm all for a deprecated.css, I feel that's a great idea and for those wanting clean installs that know they don't use any of that css hopefully they can easily dequeue it.

#6 @garrett-eclipse
11 months ago

  • Keywords needs-testing added

This is looking good as is just maybe getting someone other than myself to test the latest patch is enough to bring this to commit.

#7 @SergeyBiryukov
11 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 49213:

Administration: Remove obsolete .ac_results class references.

Core does not use the jQuery.suggest plugin since [38797].

Props afercia, hareesh-pillai, garrett-eclipse.
Fixes #40260.

Note: See TracTickets for help on using tickets.