Make WordPress Core

Opened 7 years ago

Closed 4 years ago

#40260 closed enhancement (fixed)

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

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

Description

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:
https://wordpress.slack.com/archives/C02QB2JS7/p1490370129805243

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 7 years ago.
40260.1.diff (1.0 KB) - added by Hareesh Pillai 5 years ago.
Patch refreshed
40260.2.diff (1.0 KB) - added by Hareesh Pillai 4 years ago.
Refresh
40260.3.diff (1.0 KB) - added by garrett-eclipse 4 years 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
7 years ago

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

@afercia
7 years ago

#2 @afercia
7 years ago

  • Keywords has-patch added

@Hareesh Pillai
5 years ago

Patch refreshed

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


4 years ago

#4 @SergeyBiryukov
4 years ago

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

@Hareesh Pillai
4 years ago

Refresh

@garrett-eclipse
4 years ago

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

#5 @garrett-eclipse
4 years 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
4 years 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
4 years 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.