[wp-trac] [WordPress Trac] #37233: Shiny Updates: "Add New Plugin" shiny search issues

[WordPress Trac]

#37233: Shiny Updates: "Add New Plugin" shiny search issues
-------------------------------------+-------------------------------------
 Reporter:  afercia                  |       Owner:  swissspidy
     Type:  defect (bug)             |      Status:  assigned
 Priority:  normal                   |   Milestone:  4.6
Component:  Plugins                  |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  has-screenshots shiny-   |     Focuses:  ui, accessibility,
  updates needs-patch                |  javascript
-------------------------------------+-------------------------------------
Changes (by ocean90):

 * keywords:  has-screenshots shiny-updates has-patch => has-screenshots
     shiny-updates needs-patch
 * priority:  high => normal
 * severity:  major => normal


Comment:

 Committed [attachment:37233.4.diff] with fixes for the issues mentioned in
 comment:24 in [38119].

 Using the history API is a nice idea, but not handling the back button is
 a bad idea. We'd need `$( window ).on( 'popstate', {} );` which parses the
 URL and triggers a new search... Or should we use `replaceState` instead?

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