The Needs Review Queue Bot โ tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.
- ๐ณ๐ฟNew Zealand quietone
The ideas project is being deprecated.
The issue summary refers to the problem in Seven, Bartik, and Umami themes. Since 2 of those are no longer in core and there has been several years of work on Umami, is this still relevant? Are there existing issues in core for remaining problems?
- ๐ฌ๐งUnited Kingdom catch
This was partly changed in ๐ Umami page.tpl.php breaks block placeholders Needs review which was similar but I think more problems were identified here.
@berdir just opened ๐ Umami page template renders header regions multiple times Active too.
- ๐บ๐ธUnited States kentr Durango, CO
@rinku jacob 13, ๐ Potentially remove -ms-high-contrast Needs work is probably the correct issue for your changes.
- ๐บ๐ธUnited States theMusician
I believe the ViewsUpdateConfig functions now are properly looping through existing views and setting the grouping to the default if it is not already set.
If using grouping on existing views, the default is the
element as it is already set to, after the config update executes.
- ๐บ๐ธUnited States theMusician
themusician โ changed the visibility of the branch drupal-1383696-drupal-1383696-82 to hidden.
- ๐บ๐ธUnited States theMusician
themusician โ changed the visibility of the branch drupal-1383696-drupal-1383696-82 to active.