Make WordPress Core

Opened 3 years ago

Last modified 3 years ago

#45950 new defect (bug)

Twenty Nineteen: Fix social media icons to use the correct height & width attribute

Reported by: crunnells Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 5.0.3
Component: Bundled Theme Keywords: has-screenshots needs-patch
Focuses: Cc:


This comes from a PR in GitHub: https://github.com/WordPress/twentynineteen/pull/754

The PR in question proposes to change social icon sizes to 26px, because of what _seems_ to be an erroneous setting in twentynineteen_nav_menu_social_icons(). I suggest removing the 26, since the CSS is set to show the icons at 32px, which overrides the 26px _anyway_.

Maybe update the default from 24 to 32 as well?

Attachments (2)

24px SVG icon.png (45.9 KB) - added by mukesh27 3 years ago.
32px SVG icon.png (46.8 KB) - added by mukesh27 3 years ago.

Download all attachments as: .zip

Change History (6)

#1 @mukesh27
3 years ago

@crunnells Yes theme use css of 32 height and width of SVG icon and icon render 26 social and link icon with 24.

#2 @mukesh27
3 years ago

  • Component changed from General to Bundled Theme

#3 @laurelfulford
3 years ago

  • Keywords 2nd-opinion added

Thanks @mukesh27 and @crunnells!

Just to make sure: @allancole or @kjellr, was the smaller size (24 or 26) meant to override anything in the social menu? As Chris pointed out, the CSS is setting the actual size (32); I agree that switching it to 32 in the code as well makes sense if that's the intended display size. I just want to make sure I'm not missing something here - thanks!

#4 @laurelfulford
3 years ago

  • Keywords has-screenshots needs-patch added; 2nd-opinion removed
  • Milestone changed from Awaiting Review to Future Release

Got confirmation that this can be switched to 32 in twentynineteen_nav_menu_social_icons(), so the code matches how the icons actually display.

Note: See TracTickets for help on using tickets.