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

WordPress Trac noreply at wordpress.org
Fri Jul 22 22:36:45 UTC 2016


#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 has-patch                  |  javascript
-------------------------------------+-------------------------------------

Comment (by rahulsprajapati):

 Replying to [comment:29 swissspidy]:
 > Thanks for your patch @rahulsprajapati!
 >
 > Unfortunately I don't have much time to work on this so here are some
 notes:
 >
 > * Instead of doing stuff like `bodyElem.hasClass( "plugin-install-php"
 )`, we should check the global `pagenow` variable
 > * Instead of triggering `input` events, we should put the search
 handling in a separate function and call that function. That way it's also
 better testable.
 > * What about the dropdown? That should be handled in the `popstate`
 callback too.
 > * `wp.urlParam` looks interesting. Definitely needs unit tests though.


 Added the refreshed patch. Let me know if anything else needs to update.
 :)

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


More information about the wp-trac mailing list