Content overflow issue on search form

[baroliyamayur]

Content overflow issue on search form.

Page URL:- ​https://wordpress.org/plugins/

Issue Screenshot:- ​https://share.cleanshot.com/h00rQ4497XTDDz14v2ry

[baroliyamayur]

Content overflow issue on search form

[coffee2code]

@baroliyamayur: Thanks for the report and the patch!

Your patch (here and under other tickets) includes the accumulated changes you've made to WordPress.org code for a number of different tickets (there's stuff in there for the Theme Directory, BuddyPress, Photo Directory, etc). And while it looks like you have the compiled wporg-plugins/css/style.css change that reflects your fix for this, you didn't include the .sass source file for that change, wporg-plugins/client/components/_search-form.scss.

I recommend two things for your patches going forward:

1. Check to ensure you're only including the necessary changes and changed files necessary to fix the issue in the ticket.
2. There's no need to include compiled files (.css or .js) that are generated as the result of a build process. Whoever commits the change will run the build process themselves since there will likely have been other changes in the meantime, and to ensure no other changes were snuck into the compiled files. Including the compiled files also makes the diff look much bigger than it is and harder to identify the true changes.

No worries about this ticket though, as I'm about to commit the fix.

[coffee2code]

In 12872:

Plugin Directory: Prevent excessively long search input text from flowing over search button.

Props baroliyamayur, coffee2code.
Fixes #6812.