[wp-trac] [WordPress Trac] #60548: Image editor: improve the browsePopup function

[WordPress Trac]

#60548: Image editor: improve the browsePopup function
-----------------------------+----------------------------------------
 Reporter:  afercia          |       Owner:  joedolson
     Type:  defect (bug)     |      Status:  accepted
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Media            |     Version:  6.3
 Severity:  normal           |  Resolution:
 Keywords:  has-screenshots  |     Focuses:  accessibility, javascript
-----------------------------+----------------------------------------
Changes (by joedolson):

 * owner:  (none) => joedolson
 * status:  new => accepted


Comment:

 Re: "Aside: instead of using inline events, it could have been better to
 consider a more modern approach. "

 I didn't see any reason to have everything else in inline events with just
 this event being executed differently from the rest of the image editor,
 nor did I see it as worthwhile to refactor the entire image editor. So, I
 pretty much stand by that approach in this case, even though it's far from
 ideal.

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