#Needs issue summary update

Thanks!

Wonder if this is still needed?

I'll make the call. These CSS moving efforts were worth pursuing when Drupal 8 was in development - but now that it's been around close to a decade such a change would be more disruptive than beneficial.

Wonder if this is still needed?

Since there has not been a follow up in a few years going to close out.

Just adding to the above conversation, I got around this by adding a field to media asking users to select what aspect ratio they want:

• default 16:9
• square 1:1
• vertical 4:5

I then themed as normal for my different display modes.

Changed the title, Please review.

Working on it.

This still needs a title that will have meaning when scanning the git log years from now. It is changing the API and adding a new method, after all. Something along the lines of 'Add BreakpointInterface::hasMediaQuery() for ...'?

Got another issue with this patch.
During the installation from the profile I got issue
The service "inline_entity_form.inline_block_dependency_subscriber" has a dependency on a non-existent service "node.node_route_context".

This patch should add additional dependency on the node module.

nod_ → closed merge request !7792

Committed 23301df and pushed to 11.x. Thanks!

Very close!

drpldrp → changed the visibility of the branch 2839195-10.3.x to hidden.

drpldrp → changed the visibility of the branch 2839195-10.3.x to active.

Automatically closed because Drupal 7 security and bugfix support has ended → as of 5 January 2025. If the issue verifiably applies → to later versions, please reopen with details and update the version.

The only failing test is because of 📌 Cypress test publish-validation is flaky Active

Just following up if this should be re-open,. if no follow up in 3 months could be closed

Discussed with @longwave and @catch given we're never going to upgrade 10.5.x to Twig 4 we don't need to do this one.

Looks like this needs rescoping based on #9

Given the lack of updates here I'm going to close this one out.

Please feel free to reopen with an updated issue summary including steps to reproduce if this still affects you.

Needs a reroll onto an MR.

The IS also needs an update to use the standard template.

What a fantastic start! 🤩👏

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

MR threads addressed ... so back to review.

> Update all of our API responses (both GET individual and GET list) to provide the entity operations' link relationship for those operations that are permitted

There's no individual GET for ApiContentControllers. I don't think this should be also on auto-saved data, so this should be good now?

The Ideas project is deprecated, moving this to core.

@rajveergangwar could you please fix PHP 8.4 deprecation warning in the latest patch?

Deprecated: Drupal\layout_builder\Plugin\Block\InlineBlock::__construct(): Implicitly marking parameter $logger as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/core/modules/layout_builder/src/Plugin/Block/InlineBlock.php

AFAICT 📌 Calculate field and component dependencies on save and store them in an easy to retrieve format Active would actually provide the necessary information! 🤞🤓

Hello,
Verified and Tested on Drupal 11,
MR applied successfully,
Working as expected

Steps to reproduce
1. Install Drupal 10.1.x-dev version.
2. Go to "Configuration"->Basic Site Setting-> Add Site Slogan->Save.
3. Go to Homepage and Click on the Site branding "Configure Block option and enable site slogan checkbox -> Save Block
4. Go to Homepage and observe the slogan not displayed in the site branding block.

Check attachments -
Moving this to RTBC

In previous patch, the test file was missing, so adding correct one.

📌 Manually test TUF-enabled Composer projects Active is where we’ve been getting some real-world testing

I think the memory usage looks a blocker: https://github.com/php-tuf/composer-integration/issues/127

Otherwise, we haven’t seen production issues in the last few weeks. When we do see them, they are hard to debug: https://github.com/php-tuf/composer-integration/issues/128

packages.drupal.org is TUF-enabled. For core, the drupal/* & php-tuf/* namespaces, there is packagist-signed.drupalcode.org to be added as another Composer repository. I’m guessing we might want a separate issue for adding the repository before packages.drupal.org

Should this backported?

\Drupal::$container is not initialized yet. \Drupal::setContainer() must be called with a real container.
during the cache clearing.

Got this error after patch apply and cache clearing (drush cr)
Module version: 3.0.0-rc20
Drupal version: 11.1.6

I can confirm that #115 solved the problem on Drupal core 8.9.20 (silver lining while upgrading a very out of date site).

for my case patch #118 isnt applied completaly, so i created another patch which is working fine for me.

Pretty sure this logic hasn't changed. There is an entity iterator issue around which would help with this, but a one off with a @todo for that issue also seems fine.

There is still one @todo in the MR - do we need to discuss that? I do not see it mentioned anywhere in this issue. Thanks!

Believe feedback has been addressed on this one.