[wp-trac] [WordPress Trac] #64305: Regression: Hidden async-upload input marked required in WP 6.9 RC3 prevents post publish

[WordPress Trac]

#64305: Regression: Hidden async-upload input marked required in WP 6.9 RC3
prevents post publish
-------------------------------------+-------------------------------------
 Reporter:  yagniksangani            |       Owner:  (none)
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  6.9
Component:  Posts, Post Types        |     Version:  trunk
 Severity:  major                    |  Resolution:
 Keywords:  has-patch reporter-      |     Focuses:  ui, accessibility,
  feedback                           |  administration
-------------------------------------+-------------------------------------

Comment (by wildworks):

 > Could the `required` attribute be added only `if ( 'media' ===
 get_current_screen()->id )`?

 I think whether or not it depends on the media screen seems a bit
 unstable. This may be an edge case, but try the following steps.

 - Access the legacy media uploader: http://localhost:8889/wp-admin/media-
 upload.php
 - Click the "browser uploader" link

 Since this is a JS-less upload form, the `input` element needs the
 `required` attribute to prevent form submission when no file has been
 uploaded, but with [https://github.com/WordPress/wordpress-
 develop/pull/10558 PR 10558] applied the `required` attribute disappears.

 Ideally, the `required` attribute should only be added when the HTML
 upload UI is ''visually'' displayed. In other words, the `required`
 attribute should probably be removed dynamically via JS.

 I can't think of any specific code right now, but I'm personally leaning
 towards reverting [60449] in the 6.9 release in order to implement it
 after more careful testing.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/64305#comment:21>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform