Making WordPress.org

Opened 5 weeks ago

Last modified 5 weeks ago

#8147 new defect (bug)

Support button/link has incorrect padding in active state

Reported by: takshil's profile takshil Owned by:
Milestone: Priority: low
Component: WordPress.org Site Keywords: has-patch
Cc:

Description

The "Support" link on any plugin page has incorrect padding applied when in the active state. The padding appears unevenly distributed.

Steps to Reproduce:

  1. Visit any plugin(Ex: https://wordpress.org/plugins/sync-content-from-notion/)
  2. Click on the "Support" link to activate it
  3. Observe the active state styling with excessive/incorrect padding


Attachments (1)

support-button-defect.png (11.5 KB) - added by takshil 5 weeks ago.

Download all attachments as: .zip

Change History (5)

This ticket was mentioned in PR #534 on WordPress/wordpress.org by @takshil.


5 weeks ago
#1

  • Keywords has-patch added; needs-patch removed

### Description
Fixes the Support link/button having incorrect padding on plugin pages.

Trac Ticket: https://meta.trac.wordpress.org/ticket/8147

### Problem
The "Support" link on plugin pages (e.g., [vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/code/electron-browser/workbench/workbench.html https://wordpress.org/plugins/sync-content-from-notion/]) had asymmetric padding, with the right padding missing. This caused the active state to appear visually unbalanced.

Before After
https://github.com/user-attachments/assets/4cbaf291-960f-4cb3-bf1b-479e990196d3 https://github.com/user-attachments/assets/5af39f3e-fd1d-46c6-b0e0-d226fa3fd59f

@manhphucofficial commented on PR #534:


5 weeks ago
#2

Tested on a plugin page and confirmed the Support link padding looks consistent in the active state now.

Minor suggestion: the padding could be simplified to padding: 0.64rem 1.25rem; for readability (same computed values).

#3 @manhphucofficial
5 weeks ago

Reviewed and tested the GitHub PR linked above. The fix resolves the uneven padding on the Support link active state and looks good from my testing.

Last edited 5 weeks ago by manhphucofficial (previous) (diff)

@takshil commented on PR #534:


5 weeks ago
#4

Thanks @manhphuc for looking into this. Made the change. ✅

Note: See TracTickets for help on using tickets.