Follow-up to #2916741: Don't try to include Media CSS library if Media Entity 1.x is being used → .
There is currently a @TODO to remove the form alter inside seven.theme that adds some specific CSS to media entity forms. This was pending on
#2892304: Introduce footer region to ContentEntityForm →
, which has now landed, so we can proceed to clean up this @TODO.
Use '#group' => 'footer' instead of adding specific CSS to the form.
Needs review
1.0
Code
Used to alert the usability topic maintainer(s) that an issue significantly affects (or has the potential to affect) the usability of Drupal, and their signoff is needed. When adding this tag, make it easy to review the issue. Make sure the issue summary describes the problem and the proposed solution. Screenshots usually help a lot! To get sign-off on issues with the "Needs usability review" tag, post about them in the #ux channel on Drupal Slack, and/or attend a UX meeting to demo the patch and get direct feedback from designers/UX folks/product management on next steps. If an issue represents a significant new feature, UI change, or change to the general "user experience" of Drupal, use Needs product manager review instead. See the scope of responsibilities for product managers.
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
So looking at the IS and comments this appears to be specific to seven to me? Which was removed in D10
If you disagree please move back with an updated issue summary
Thanks