Name:
Layout builder admin theme

Description:
This module forces a Drupal site to use the admin theme when using layout builder.
Sometimes a custom front end theme makes using layout builder very difficult. This problem can be fixed by using the admin theme instead. Layout builder admin theme is a lightweight module that solves this problem.
It has a single setting to toggle the admin theme on or off when using layout builder.
All code in the module meets Drupal coding standards.
Drupal 8 or 9 compatible.

Project link

https://www.drupal.org/project/layout_builder_admin_theme

Git instructions

git clone --branch 8.x-1.x https://git.drupalcode.org/project/layout_builder_admin_theme.git

CommentFileSizeAuthor
3155484-9.patch70 bytesridhimaabrol24

Comments

robert.duplock created an issue. See original summary.

avpaderno’s picture

Title: [D8, D9] Layout builder admin theme » [D8] Layout builder admin theme
Issue summary: View changes
avpaderno’s picture

Thank you for applying! Remember to change status, as in this issue queue, Active is used when the project is not jet ready to be reviewed. When the project is ready to be reviewed, the status should be set to Needs review.

robert.duplock’s picture

Status: Active » Needs review
sadashiv’s picture

I tried to review this and the module does what it says:
1) Without the module installed the layout page for book content type admin/structure/types/manage/book/display/default/layout showed frontend theme and with the module enabled it showed admin theme
2) Also the configuration works fine i.e. turning off in admin menu shows the frontend theme again.
3) PHPCS shows no errors, etc.
4) README looks good.

I think it's all good.

Thanks,
Sadashiv

shaktik’s picture

looks good to me, PHPCS shows no errors.

harishh’s picture

No errors found and also maintained the Drupal standards.

phthlaap’s picture

Status: Needs review » Needs work

Please capitalize the name of the module and dependency namespacing in .info.yml file

Link to follow: https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...

avpaderno’s picture

Status: Needs review » Needs work

The task of reviewers is pointing out what needs to be changed, not providing patches. We don't review patches, and the status is changed only when the user who is applying commits the changes in the project repository, or the described changes are wrong.

robert.duplock’s picture

Status: Needs work » Needs review
5droid’s picture

If I am using Drupal 7 for my website. Do I need to upgrade to Drupal 8 or 9?

avpaderno’s picture

@5droid This is not the place to ask for support. It's the review queue for users who want to get the vetted role.

avpaderno’s picture

Priority: Normal » Critical

I am changing priority as per https://www.drupal.org/node/539608.

phthlaap’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for your application, I don't see any security issue related into the code.

klausi’s picture

Status: Reviewed & tested by the community » Fixed

Reviewed the module, all looking good to me.

Thanks for your contribution, Rob!

I updated your account so you can opt into security advisory coverage now.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on Slack or IRC in #drupal-contribute. So, come hang out and stay involved!

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

Thanks to the dedicated reviewer(s) as well.

klausi’s picture

Assigning credits.

robert.duplock’s picture

Many thanks to everyone who took the time to help with this security coverage application.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.