Olivero: Focus after submenu close via ESC key
Discovered in #3190120: Olivero: Focused level-2 nav items should always be in viewport during keyboard navigation.If a dropdown menu is open, and ESC is pressed, focus should return to the dropdown...
View Articlefield_language shouldn't cache display_languages for new entities
The field_language function according to its documention: 'Returns the display language for the fields attached to the given entity.'It caches its internal results per entity type and entity id (per...
View Articledrupal/core-php-storage should use restrictive file permissions
Problem/MotivationWe shouldn't allow open-access to the compiled php container. Previously we'd had strict permissions on the sites/default/files/php folder. Now it is getting 777 permissions.Proposed...
View ArticleResponsive image field formatter list should be sorted by label, not machine...
Problem/MotivationCurrently when configuring a responsive image format for a field, the list of responsive styles are sorted by machine name, not the style label. This can be confusing if the style...
View ArticlecalculateDependencies doesn't check $mode_entity if it is NULL
Problem/Motivation1. Created a feature exporting user field settings with the contributed D8 feature module 2. Various changes to the user fields 3. Incorporating my colleagues various changes into my...
View ArticleClik here to view.
Olivero: Quickedit in Tabledrag has overflow
Split off from #3191725: Ensure that tabledrag user interface is usable within OliveroNote this is also somewhat broken in Bartik. Before patch:After patch:
View ArticleContent translation overview should add destination query parameter
Problem/MotivationIt would be helpful when you're on the translation overview to return to the overview after adding or editing the translation. This matches what we do for content listings.Proposed...
View ArticleNon-exposed and exposed sorts cause issues with the default sort order
Problem/MotivationSo the issue started when I added a non-exposed sort as the first sort out of three. The other two sorts are exposed, which looks something like this:Enabled: boolean ASC...
View ArticleClik here to view.
Enable Claro as the admin theme in Umami
Problem/MotivationNow that Claro has been added to Drupal core in #3079738: Add Claro administration theme to core, we can make it the default admin theme in the Umami demo profile.Proposed resolutionI...
View ArticleMissing schema for weight field in Media Library widget
Problem/MotivationWhen setting default value in entity reference field, which reference media and using media library widget, in configuration there is `weight` field, which is not described in any...
View ArticleProvide an Olivero sub-theme creation tool
Problem/MotivationSite builders often look for an easy way to create a theme, and sub-theming is a great solution, for example based on Bootstrap.Proposed resolutionIt would be nice if a sub-theme...
View ArticleReduce cache variations in locale cache
Problem/Motivation\Drupal\locale\LocaleLookup::getCid uses user-roles cache id. User roles have no specific order, it depends on which order the user got the roles. The cache key should be identical...
View ArticleEntity storage exception during module install missing !message parameter in...
Problem/MotivationIf there is an entity storage exception thrown during module install, the watchdog_exception() is missing a replacement for the !message placeholder, so the exception message is not...
View ArticleSeveral IF with confusing conditions, as "a OR b AND c" without ( )
Confusing conditions structure:# @file /core/modules/views/src/Plugin/views/style/Table.php, line 187 if ($field == $column || $columns[$column] == $column && !empty($sanitized[$column])) { #...
View ArticleUse the user display name instead of the username on the user authoring...
Problem/MotivationWhen editing a node, the authoring information displays the user account name, the display name should be used instead.Steps to reproduceIn an installation with some module that...
View ArticleText item sample generation fails if max length < 3
Problem/MotivationWhen the maximum length of a text field is set to less than 3 the generateSampleValue() function will throw an error:mt_rand(): max(0) is smaller than min(1) This is because the max...
View ArticleDeprecate and replace jQuery Joyride (for tours)
Problem/MotivationjQuery Joyride was added to support tours. It is used in a Views tour and a welcome tour in Umami. However Joyride has not been maintained for the last 2 years. See...
View ArticleClik here to view.
Blockquote can overflow into the sidebar in the Olivero theme
Problem/Motivationblockquote and pre can overflow into the right sidebar region in the Olivero themeSee screenshot:Steps to reproduce1. Create a site e.g. with...
View ArticleUnable to assign roles to the admin account
Problem/MotivationNot possible to assign a role to the admin account.Steps to reproduceInstall a fresh Drupal 9 site.Select a language other than English.Standard profileAdd a role.Try to assign that...
View ArticleClik here to view.
Add an implementation of EntityDisplayBase::label()
Problem/MotivationNeither EntityFormDisplay nor EntityViewDisplay have a label key, and as such their ::label() method returns NULL.Proposed resolutionImplement ::label(), returning a string indicating...
View Article