Make WordPress Core

Opened 7 weeks ago

Last modified 6 weeks ago

#64143 reopened enhancement

Plugin search box lacks margin

Reported by: jilani53's profile jilani53 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Plugins Keywords: changes-requested has-patch
Focuses: css, administration Cc:

Description (last modified by sabernhardt)

The UI appears fine with the default theme or when WooCommerce is inactive, as there’s no additional menu item under “Search Plugins.” However, when WooCommerce is activated, it adds a new menu item called “WooCommerce Marketplace.” After activation, the plugin search interface looks cluttered and less professional.

https://prnt.sc/w5wj3PTzhfar

Attachments (4)

search-plugin.png (67.9 KB) - added by jilani53 7 weeks ago.
Issue screenshot
add-plugins-search-results-1150px.png (27.4 KB) - added by sabernhardt 7 weeks ago.
Add Plugin screen at 1150 pixels wide, with search box below six filter links (including Search Results and Beta Testing)
add-plugins-search-results-fr-1324px.png (29.7 KB) - added by sabernhardt 7 weeks ago.
French plugin search results, without Beta Testing link, at 1324 pixels wide
64143.diff (789 bytes) - added by jilani53 6 weeks ago.
Fix plugin search form

Download all attachments as: .zip

Change History (14)

@jilani53
7 weeks ago

Issue screenshot

#1 @jilani53
7 weeks ago

Hello,

I accidentally uploaded a sensitive file as an attachment:

https://core.trac.wordpress.org/attachment/ticket/64143/search-plugin.png

Could a Trac moderator please remove this file as soon as possible for security reasons?

Additionally, please update the ticket description to replace the screenshot with this one instead:
https://prnt.sc/w5wj3PTzhfar

Thank you very much for your help!

Last edited 7 weeks ago by jilani53 (previous) (diff)

#2 @jorbin
7 weeks ago

  • Focuses ui css removed
  • Keywords needs-design removed
  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed
  • Version trunk deleted

Hi @jilani53, welcome to WordPress core Trac. This trac is used for the development of the WordPress software, not for assistance with individual sites or plugins.

As the issue you are experiencing is caused by the WooCommerce plugin, I would suggest reaching out to the WooCommerce Support Forums for assistance.

I've closed this ticket of with the term invalid, in this case it's the unfriendly term trac uses to indicate that I've referred you to somewhere where you can hopefully find people better able to assist you.

@sabernhardt
7 weeks ago

Add Plugin screen at 1150 pixels wide, with search box below six filter links (including Search Results and Beta Testing)

@sabernhardt
7 weeks ago

French plugin search results, without Beta Testing link, at 1324 pixels wide

#3 @sabernhardt
7 weeks ago

  • Component changed from General to Plugins
  • Description modified (diff)
  • Focuses css administration added
  • Milestone set to Awaiting Review
  • Resolution invalid deleted
  • Status changed from closed to reopened
  • Summary changed from Some visual adjustments could enhance usability. to Plugin search box lacks margin

#61785 made an adjustment, but the search box still can wrap under the filter links on screens wider than 1138px.

With the Search Results and Beta Testing links, the search box drops beneath the links for me, without the top and bottom margin, at 1212 pixels wide in English.

In French, the search form dropped at 1324px when I had Search Results, without the additional link for Beta Testing.

@jilani53
6 weeks ago

Fix plugin search form

#4 @jilani53
6 weeks ago

Thanks for testing and the feedback!

I think we don't need an extra media screen for this.
We can solve the wrapping issue by adding a global margin to the ".wp-filter .search-form" class, as shown in my attached patch 64143.diff.

Here is the result after updating the CSS.

https://prnt.sc/cnAIDaIRvm1t

Last edited 6 weeks ago by jilani53 (previous) (diff)

#5 @jilani53
6 weeks ago

Hi @jorbin, just to clarify:

This ticket and the attached patch are related to WordPress core admin CSS (src/wp-admin/css/common.css) and not any plugin.

The patch fixes the wrapping issue of the search box in the admin list tables by adding a global margin to the ".wp-filter .search-form" CSS class.

This change only affects WordPress core styles and does not depend on WooCommerce or any other plugin.

Here is the "Plugin search box lacks margin" issue's solution - https://prnt.sc/cnAIDaIRvm1t

Last edited 6 weeks ago by jilani53 (previous) (diff)

#6 @nikunj8866
6 weeks ago

  • Keywords changes-requested added

@jilani53 Please update the CSS as follows:

.wp-filter .search-form {
    margin: 10px 0px;
}

The current:

.wp-filter .search-form {
    margin: 10px;
}

creates unnecessary spacing on the left and right sides.

@jilani53 If it is possible for you to apply your latest patch to a new pull request to our GitHub mirror, that will help in making the review process smoother. You can find the instructions here: https://make.wordpress.org/core/handbook/contribute/git/github-pull-requests-for-code-review/.

#7 @jilani53
6 weeks ago

@nikunj8866 For your kind attention - after applying your updated code, the alignment doesn’t look quite right.
Here’s a screenshot for reference: https://prnt.sc/06X4EEMyxYfb

I think adding a 10px margin on all sides would help keep the alignment and spacing looking right.

This ticket was mentioned in PR #10414 on WordPress/wordpress-develop by @jilani53.


6 weeks ago
#8

  • Keywords has-patch added

#9 @sabernhardt
6 weeks ago

  1. The .wp-filter .search-form selector also applies on the Media Library screen, and it should not have the extra margin in List mode.
  2. A left margin can help when the search form is below the filter links, but the form does not need a right margin when it displays to the right of the filters.

An additional ruleset can be more specific, placed before .wp-filter .search-form.search-plugins select:

.wp-filter .search-form.search-plugins {
	margin: 11px 0 11px 10px;
}

#10 @jilani53
6 weeks ago

We should also add a right margin to maintain proper right alignment — https://prnt.sc/hcw-CAs_m-7M

This overall adjustment eliminates the need for additional media queries.

Last edited 6 weeks ago by jilani53 (previous) (diff)
Note: See TracTickets for help on using tickets.