WordPress.org

Make WordPress Core

Opened 5 weeks ago

Closed 2 weeks ago

#49302 closed defect (bug) (fixed)

Theme action button container is out of align by 1px

Reported by: ideaboxcreations Owned by: audrasjb
Milestone: 5.4 Priority: normal
Severity: normal Version: 5.3
Component: Themes Keywords: good-first-bug has-screenshots has-patch commit
Focuses: ui, css, administration Cc:

Description

Theme action button container is 1px larger than its parent container. It can be fixed by reducing padding.

Steps to check:

  1. Go to WP admin > Appearance > Themes
  2. Check the action button container

I've attached a screenshot for reference.

Attachments (5)

Screenshot 2020-01-27 at 4.16.06 PM.png (31.0 KB) - added by ideaboxcreations 5 weeks ago.
See the highlighted area.
Screenshot 2020-01-27 at 4.17.28 PM.png (51.0 KB) - added by ideaboxcreations 5 weeks ago.
Zoomed a little bit in case if not visible
49302.diff (123.9 KB) - added by aftabmuni 3 weeks ago.
Reduced padding of button container by 1px.
49302_1.diff (737 bytes) - added by aftabmuni 3 weeks ago.
Removed changes from minified files.
49302.2.diff (681 bytes) - added by alexischenal 3 weeks ago.
New refreshed patch

Download all attachments as: .zip

Change History (15)

@ideaboxcreations
5 weeks ago

See the highlighted area.

@ideaboxcreations
5 weeks ago

Zoomed a little bit in case if not visible

#1 @audrasjb
5 weeks ago

  • Milestone changed from Awaiting Review to 5.4
  • Owner set to audrasjb
  • Status changed from new to accepted
  • Version changed from trunk to 5.3

Welcome to WordPress Trac @ideaboxcreations and thank you for opening this ticket,

I can reproduce the issue on both Chrome and Firefox / Mac OS.
Moving to milestone 5.4.

#2 @audrasjb
5 weeks ago

  • Keywords has-screenshots added

This could be fixed either by using a transparent background on .theme-actions div element, or different padding values.

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

#3 @Presskopp
3 weeks ago

  • Keywords good-first-bug added

@aftabmuni
3 weeks ago

Reduced padding of button container by 1px.

#4 @aftabmuni
3 weeks ago

  • Keywords has-patch added; needs-patch removed

#5 @Presskopp
3 weeks ago

You don't need to patch the minified versions...

@aftabmuni
3 weeks ago

Removed changes from minified files.

#6 follow-up: @audrasjb
3 weeks ago

  • Keywords needs-refresh added

Thanks for your contribution @aftabmuni !
RTL files are also automatically generated. I'll test your proposal and make sure to refresh the patch from trunk in the next few hours. This ticket is still on track for beta 1.

#7 in reply to: ↑ 6 @aftabmuni
3 weeks ago

Oh So Sorry.

Replying to audrasjb:

Thanks for your contribution @aftabmuni !
RTL files are also automatically generated. I'll test your proposal and make sure to refresh the patch from trunk in the next few hours. This ticket is still on track for beta 1.

#8 @audrasjb
3 weeks ago

No worries @aftabmuni It's nearly alright, just need few checks and refresh before commit ;-)

@alexischenal
3 weeks ago

New refreshed patch

#9 @audrasjb
3 weeks ago

  • Keywords commit added; needs-refresh removed

Thanks @alexischenal I tested the latest patch and it looks good to go on my side, marking this for commit.

Last edited 3 weeks ago by audrasjb (previous) (diff)

#10 @SergeyBiryukov
2 weeks ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 47293:

Themes: Correct the height of theme actions button container on various screen sizes.

Props aftabmuni, alexischenal, ideaboxcreations, audrasjb, SergeyBiryukov.
Fixes #49302.

Note: See TracTickets for help on using tickets.