WordPress.org

Make WordPress Core

Opened 3 weeks ago

Last modified 2 weeks ago

#47369 new defect (bug)

Admin pages have two viewport meta tags on mobile

Reported by: BettyJJ Owned by:
Milestone: 5.3 Priority: normal
Severity: normal Version: 3.3
Component: Administration Keywords: needs-patch
Focuses: Cc:

Description

One is add by wp-admin/admin-header.php:

<meta name="viewport" content="width=device-width,initial-scale=1.0">

Another one is added by wp-admin/includes/admin-filters.php and the content is

<meta name="viewport" id="viewport-meta" content="width=device-width, initial-scale=1">

Use Chrome DevTools to emulate any mobile browser and you can see them.

Having two viewport meta tags is not good practice. Besides, there is really no need to use two viewport meta tags here since their contents are virtually identical.

Change History (4)

#1 @mukesh27
3 weeks ago

  • Keywords dev-feedback added
  • Version set to 3.3

Hi @BettyJJ, welcome to WordPress Trac! Thanks for the report.

Also replicated same issue in mobile device.

Second viewport added via wp-admin/includes/admin-filters.php file with add_action( 'admin_head', '_ipad_meta' ); add_action code and it was introduce in WordPress 3.3.0

/**
 * @since 3.3.0
 */
function _ipad_meta() {
	if ( wp_is_mobile() ) {
		?>
		<meta name="viewport" id="viewport-meta" content="width=device-width, initial-scale=1">
		<?php
	}
}

@desrosj can you please review above and add your thought or further guidance.

#2 @SergeyBiryukov
3 weeks ago

  • Component changed from General to Administration
  • Keywords needs-patch added; dev-feedback removed
  • Milestone changed from Awaiting Review to 5.3

_ipad_meta() introduced in [18926], global <meta name="viewport"> introduced in [26134].

Looks like _ipad_meta() can be deprecated now.

#3 @BettyJJ
3 weeks ago

I'm glad my report is received so quickly. Thank you.
Between the two, I personally like _ipad_meta()'s add_action approach since it allows users to change or remove it if they need. So, whichever you'd keep, please consider using add_action instead of hardcoding it in admin-header.php. Thank you. 😊

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

#4 @ajayghaghretiya1
2 weeks ago

If _pad_meta() will be deprecated then do we need to remove all the dependency as in [18926]?

Thanks.

Note: See TracTickets for help on using tickets.