[PP-?] Show revision state on the revisions overview tab for moderated...
Problem/MotivationWhen Content Moderation module is enabled, different revisions of a node each have their own moderation state, e.g.:- r5: draft - r4: draft - r3: published - current revision - r2:...
View Article[meta] Requirements for tagging Drupal 10.0.0-beta1
Must-haves prior to tagging 10.0.0-beta1Fully support PHP versions above the Drupal 10 minimum, including PHP8.2 when adding support becomes possible. PHP 8.2: #3283358: [META] Make Drupal 9/10...
View ArticlegetEntityTypeLabels uses translatable strings as array keys
Problem/MotivationTranslatable strings shouldn't be used as array keys. They can easily be changed from the UI causing things to break. In Drupal\Core\Entity\EntityTypeRepository::getEntityTypeLabels()...
View ArticleFix default value of author in \Drupal\comment\CommentStatistics::create()
Problem/MotivationFound this bug while review #2318875-20: Redo CommentStatisticsInterface This broken at least from #2101183: Move {comment_entity_statistics} to proper serviceProposed resolutionFix...
View ArticleExecute hook_block_view_alter when placing blocks with layout builder
Problem/MotivationHook block alter does not execute for blocks that have been placed in Layout builder which means layout builder displays blocks incorrectly without their alters from contrib. Here's...
View ArticleRemove modernizr usages from core
Problem/MotivationWe want to deprecate and then remove modernizr in core.In order to do that, we need to make sure core isn't reliant on it any more. This includes any JavaScript dependencies on it, as...
View ArticleDecide what to do about CKEditor styles in core themes
Problem/MotivationCore themes (Seven, Claro, Umami, Olivero and Bartik) ship with CSS that is loaded inside the CKEditor iframe. Since CKEditor 4 is being deprecated, we need to decide what to do about...
View ArticleOlivero: Rename "wide-image" CSS class to "wide-content"
Within Olivero, we use a wide-image CSS class to pull the article's image to the left and then make it wider. However, this class can also be used for embedded content within the body text (I'm doing...
View Article[PP-2] Let GDToolkit support AVIF image format
This issue is currently postponed on:a bot with a PHP 8.1. instance that can fully support AVIF files - currently the bot PHP cannot save files via imageavif(), a codec seems missing -...
View Articlehook_theme() docs for path are inaccurate
API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21...The docs for this say:path: Override the path of the file to be used. Ordinarily the module or theme path will be...
View ArticleMedia Library Tests should not rely on Classy
Problem/MotivationBecause of #3083275: [meta] Update tests that rely on Classy to not rely on it anymore and Classy being deprecated in Drupal 9 + removed in Drupal 10,: Tests that aren't specifically...
View ArticleMedia Tests should not rely on Classy
Problem/MotivationBecause of #3083275: [meta] Update tests that rely on Classy to not rely on it anymore and Classy being deprecated in Drupal 9 + removed in Drupal 10,: Tests that aren't specifically...
View ArticleEditing source using the CKEditor Source mode does not fire change event
Problem/MotivationThe Editor module tracks when an editor has modified a field, and reverts the value of a field when its editor has not made any changes. When editing HTML source in CKEditor, the...
View ArticleAdd vendor-update command to the build command for JS assets
Problem/MotivationTo prevent issues with vendor assets when updating things like terser, add the vendor-update to the build command. See #3296481-32: Update terser and terser-webpack-plugin to the...
View ArticleAllow multiple recipients in email actions
It is not possible at the moment to send an email to multiple recipients without assigning multiple actions to a trigger. This patch allows a comma separated list of email addresses to be used, rather...
View ArticleMigrationInterface::mergeProcessOfProperty() doesn't document who wins
Problem/MotivationThe docs say: * Merge the process pipeline configuration for a single property. * * @param string $property * The property of which to merge the passed in process pipeline *...
View ArticleExtensionDiscovery documentation is wrong about scanning profile directories
Problem/MotivationThe \Drupal\Core\Extension\ExtensionDiscovery class documentation implies that you can control whether or not extensions underneath a profile directory will be scanned or not. But...
View ArticleUpgrade HTML::escapeCdataElement to correctly comment CDATA
Problem/MotivationAs of now, HTML::escapeCdataElement uses probably old way of commenting cdata element. For example, for cdata inside script tag the output is<!--//--><![CDATA[//...
View ArticleAllow AJAX links to replace a specific selector
Problem/MotivationThis is a follow-up of #3020716: Add vertical tabs style menu to media library. The current AJAX system supports opening dialogs by adding the use-ajax class to a link. In the media...
View ArticleAdd a token for the site logo
Problem/MotivationSpun off from this request for Drupal 7 (token module): #823780: Add site logo token, where a token for the logo and logo-url have been proposed. However, since all site:* tokens are...
View Article