Olivero: Show content preview checkbox is not center aligned with the layout builder buttons.

@shweta__sharma MR#2 applied successfully.

I'm triaging RTBC issues → . I read the IS and the comments. I didn't find any unanswered questions or other work to do. It took me a while to figure out that the image in the 'user interface changes' section of the issue summary is the before image not the after image.

For issues that are changing the UI it is helpful to have the before and after images in the issue summary, or links to the latest before and after screenshots.

Leaving at RTBC.

I have tested merge request !5573, and here are my observations: In the case of the sidebar, the checkbox item is wrapped onto the next line, and the spacing is increased even if we don't want that, as the alignment is already fine on the next line. I have attached a screenshot for reference.

Gauravvvv → changed the visibility of the branch 3404663-olivero-show-content to hidden.

I have provided a separate patch with a different approach. I'm not updating the current merge request, so the maintainer can choose any approach they prefer.

If a different solution is going to be used it should be reflected in the issue summary. #11 doesn't mention what the new solution is.

A separate MR should be opened and in the issue summary have something like

Option A

Option B

So discussion can be had about best approach.

Created new branch and MR for second solution also updated Issue summary. Please review.

Added both the approaches in the IS with the MR link.

Thanks

MR has failures.

aayushmankotia → changed the visibility of the branch 3404663-content-preview-checkbox-alignment to hidden.

aayushmankotia → changed the visibility of the branch 3404663-content-preview-checkbox-alignment to hidden.

ahsannazir → made their first commit to this issue’s fork.

aayushmankotia → changed the visibility of the branch 3404663-content-preview-checkbox-alignment to active.

aayushmankotia → changed the visibility of the branch 3404663-olivero-show-content to active.

Hi
I've tested MR !6747 on Drupal 11.x
The patch is applied successfully...

my observations:
The checkbox is center-aligned with the layout builder buttons.
In the case of the sidebar- The checkbox wrapped onto the next line, Spacing is not changed.

Adding Screenshots

Hi, verified and tested MR !6447 and applied Patch successfully on 11.x version. And all the changes are working as expected.

Following steps I followed:
For scenario 1 "Checkbox alignment"
1. Open local and Install Oliverio theme
2. Go to /admin/structure/types/manage/article/display/default/layout
3. Checked the show content preview checkbox and its alignment and observed the changed here

Testing Result for Scenario 1:
The Alignment of the "Show content preview" looks proper as expected.
Attaching the SS for reference.

For Scenario 2 " When sidebar is added spacing issue"
1. Go to /admin/structure/types/manage/article/display/default/layout
2. Added the Sidebar in the Layout
3. Now observed the changes here

Testing result for Scenario 2:
The spacing looks as expected when we add the side bar and "Show content preview" Checkbox displayed in next line
Attaching the SS for reference.

Hence moving to RTBC!
Thanks

nod_ → closed merge request !5573

nod_ → closed merge request !6747

went with align:middle since it works and I don't want to add yet another variable.

Committed 572e327 and pushed to 11.x. Thanks!

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