[wp-trac] [WordPress Trac] #46456: Customize: widgets search shouldn't search by widgets id

WordPress Trac noreply at wordpress.org
Sun Dec 15 04:01:17 UTC 2019


#46456: Customize: widgets search shouldn't search by widgets id
---------------------------------------+---------------------
 Reporter:  afercia                    |       Owner:  (none)
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  5.4
Component:  Customize                  |     Version:  3.9
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:  ui
---------------------------------------+---------------------
Changes (by dlh):

 * keywords:  has-screenshots => has-screenshots has-patch
 * milestone:  Awaiting Review => 5.4


Comment:

 I'm not sure a widget ID could be changed without backwards-compatibility
 considerations, but the given reasons generally make sense to me. The IDs-
 can't-be-translated screenshots seem persuasive by themselves.

 However, there's similar code in place for searching themes, which also
 considers the theme slug, or `id`. The slug support was added quite
 deliberately in #26730.

 I don't have the context to say whether ID support was added to widgets
 for parity with theme searches. But I could see an argument that site
 owners are somewhat more likely to come into contact with a theme slug
 than with the ID of a widget in the codebase, and so including IDs makes
 sense in one case but not the other.

 [attachment:"46456.diff"] is in that spirit. Perhaps a decision could be
 made on this suggestion for 5.4?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/46456#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list