[wp-trac] [WordPress Trac] #59067: Comment text regarding wp_min_priority_img_pixels filter in wp_get_loading_optimization_attributes function

WordPress Trac noreply at wordpress.org
Fri Aug 11 06:19:04 UTC 2023


#59067: Comment text regarding wp_min_priority_img_pixels filter in
wp_get_loading_optimization_attributes function
--------------------------+-----------------------------
 Reporter:  tmatsuur      |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Media         |    Version:  6.3
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-----------------------------
 The wp_min_priority_img_pixels filter portion of the
 wp_get_loading_optimization_attributes function is described as follows

 {{{
 /** This filter is documented in wp-admin/includes/media.php */
 $wp_min_priority_img_pixels = apply_filters( 'wp_min_priority_img_pixels',
 50000 );
 }}}

 I looked in wp-admin/includes/media.php and there is no mention of this
 filter.

 Instead, I found a description of this filter in the
 wp_maybe_add_fetchpriority_high_attr function in wp-includes/media.php.

 Is it possible that the wp_maybe_add_fetch_priority_high_attr function has
 been changed from wp-admin/includes/media.php to wp-includes/media.php?

 If so, that would explain why there was "wp-admin/includes/media.php" in
 the comment text.

 I think the following comment text is correct


 {{{
 /** This filter is documented in wp-includes/media.php */
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/59067>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list