Spacing issue between Checkbox label and button

Created on 22 July 2024, 5 months ago
Updated 10 September 2024, 3 months ago

Problem/Motivation

Steps to reproduce

Step1 : Fresh install Drupal 11 and use default Olivero theme.
Step2 : Enable layout builder module
Step3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options
Step4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.

See attached screen shot for reference.

Proposed resolution

Add some spacing on right side of "Show content preview" label.

Remaining tasks

Review

User interface changes

Before

After

Introduced terminology

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

🐛 Bug report
Status

RTBC

Version

11.0 🔥

Component
CSS 

Last updated 8 days ago

Created by

🇮🇳India jaydeep_patel Ahmedabad

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @jaydeep_patel
  • Issue was unassigned.
  • Status changed to Needs review 5 months ago
  • 🇮🇳India jaydeep_patel Ahmedabad

    Spacing issue between Checkbox label and button has been fixed.

  • Pipeline finished with Success
    5 months ago
    Total: 684s
    #231269
  • Status changed to Needs work 5 months ago
  • The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. 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

    Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to our policies.

  • Assigned to yujiman85
  • 🇺🇸United States yujiman85

    This looks like it needs a rebase so I can take care of that.

  • Issue was unassigned.
  • Status changed to Needs review 4 months ago
  • 🇺🇸United States yujiman85

    Branch is updated and just needs a review.

  • Pipeline finished with Success
    4 months ago
    Total: 661s
    #265174
  • Reviewed Issue is fixed.

    Steps to reproduce.
    1 : Fresh install Drupal 11 and use default Olivero theme.

    2 : Enable layout builder module

    3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options

    4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.
    5. Issue is fixed with give MR.

  • Status changed to RTBC 4 months ago
  • 🇺🇸United States smustgrave

    Completed the issue summary.

    Can confirm the issue on 11.x and that the MR addresses the problem.

  • Status changed to Needs work 4 months ago
  • 🇫🇷France nod_ Lille
  • Status changed to Needs review 4 months ago
  • 🇺🇸United States yujiman85

    I moved the change into the Olivero theme and targeted the ID for that div. Needs a review.

  • Pipeline finished with Failed
    4 months ago
    Total: 465s
    #266361
  • Pipeline finished with Failed
    4 months ago
    Total: 278s
    #266366
  • As per previous comments changes are now moved to Olivero theme and targeted the ID for that div.
    I can confirm on 11.x updated MR addresses the problem. https://www.drupal.org/project/drupal/issues/3463111#comment-15746322 🐛 Spacing issue between Checkbox label and button Needs review

    Adding screenshot for reference.

    Steps to reproduce.
    1 : Fresh install Drupal 11 and use default Olivero theme.

    2 : Enable layout builder module

    3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options

    4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.
    5. Issue is fixed with give MR.

  • 🇮🇳India sagarmohite0031

    Hello
    I have reproduced the issue for Spacing issue between Checkbox label and button on Drupal 11.
    The MR is applied successfully.

    Steps to reproduce
    Step1 : Fresh install Drupal 11 and use default Olivero theme.
    Step2 : Enable layout builder module
    Step3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options
    Step4 : Create Basic page from Content > Add content > Basic page and Save
    Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.

    See attached screen shot for reference.

    Test Result:
    Spacing issue between Checkbox label and button has been fixed.
    Check attachments.

    RTBC+1

  • 🇺🇸United States smustgrave

    Restoring tags

  • Status changed to Needs work 4 months ago
  • 🇺🇸United States smustgrave

    MR appears to have an issue.

    FYI when making a css change have to edit the pcss first and recompile.

  • 🇮🇳India sanket.tale

    sanket.tale made their first commit to this issue’s fork.

  • Status changed to Needs review 4 months ago
  • 🇬🇧United Kingdom rachel_norfolk UK

    I've moved to Needs Review but I'll be leading some new contributors tomorrow through things and this would be a great one for them to check. Can you leave any review until at least the weekend? Thanks!

  • Pipeline finished with Failed
    4 months ago
    Total: 4862s
    #274638
  • 🇵🇭Philippines binssss Bacoor Cavite

    Hi! We'll be reviewing this change today as a group; this is our first time contributing.

  • 🇵🇭Philippines rduterte PH

    Hello, We're going to be testing this changes for our first contribution.

  • Pipeline finished with Success
    4 months ago
    Total: 545s
    #275376
  • 🇵🇭Philippines binssss Bacoor Cavite
  • Status changed to RTBC 4 months ago
  • Status changed to Needs work 4 months ago
  • 🇫🇷France nod_ Lille

    Thanks, better.
    We try our best to not use IDs in CSS, so have a look at how umami solves this issue and apply the same to Olivero/Claro.
    The problem happens also on Claro, so that needs fixing there as well.

  • Pipeline finished with Success
    4 months ago
    Total: 817s
    #275685
  • Status changed to Needs review 3 months ago
  • Status changed to Needs work 3 months ago
  • 🇫🇷France nod_ Lille

    Nice, Claro also need the fix

  • I have fixed Spacing issue between Checkbox label and button for Claro theme.

    Adding screenshot for reference.

    Steps to reproduce.
    1 : Fresh install Drupal 11 and use default Claro theme.

    2 : Enable layout builder module

    3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options

    4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.

  • Pipeline finished with Failed
    3 months ago
    Total: 2301s
    #278044
  • Pipeline finished with Failed
    3 months ago
    Total: 635s
    #278074
  • Pipeline finished with Success
    3 months ago
    Total: 431s
    #278088
  • Status changed to Needs review 3 months ago
  • Status changed to RTBC 3 months ago
  • 🇺🇸United States smustgrave

    Restoring status.

  • 🇳🇿New Zealand quietone

    I read the IS, comments, and the MR. There are no unanswered questions. I did not test this.

    Leaving at RTBC.

  • 🇫🇷France nod_ Lille

    question in MR

  • 🇬🇧United Kingdom rachel_norfolk UK

    A quick update to the MR to resolve issues mentioned by Nod_ and keep the issue rolling forwards.

    I’ve updated the test steps a little to say that the test needs to be performed both in Olivero and Claro.

    Of course, this now needs the manual tests re-running to ensure that the change applies to BOTH Claro and Olivero. So, four screenshots. That is, once again, a perfect Novice task...

  • Pipeline finished with Failed
    2 months ago
    Total: 560s
    #305284
  • 🇮🇳India sagarmohite0031

    Hello
    I have reproduced the issue for Spacing issue between Checkbox label and button on Drupal 11.
    The MR is applied successfully.

    Steps to reproduce
    Step1 : Fresh install Drupal 11 and use default Olivero theme.
    Step2 : Enable layout builder module
    Step3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options
    Step4 : Create Basic page from Content > Add content > Basic page and Save
    Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.

    See attached screen shot for reference.

    Test Result:
    Spacing issue between Checkbox label and button has been fixed.
    Check attachments.

  • 🇬🇧United Kingdom rachel_norfolk UK

    Awesome, thanks Sagar. Can you repeat the test looking at the page via the Claro theme? Then, if successful, set the status to Reviewed and Tested by the Community?

  • 🇮🇳India sagarmohite0031

    Hello rachel_norfolk,
    I have reproduced the issue for Spacing issue between Checkbox label and button on Drupal 11 for Claro.
    The MR is applied successfully.

    Steps to reproduce
    Step1 : Fresh install Drupal 11 and use default Claro theme.
    Step2 : Enable layout builder module
    Step3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options
    Step4 : Create Basic page from Content > Add content > Basic page and Save
    Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.

    See attached screen shot for reference.

    Test Result:
    Spacing issue between Checkbox label and button has been fixed.
    Check attachments.

    • nod_ committed bb1fad91 on 10.3.x
      Issue #3463111 by sanket.tale, jaydeep_patel, yujiman85, sheetal.pathak...
    • nod_ committed 31afe9da on 10.4.x
      Issue #3463111 by sanket.tale, jaydeep_patel, yujiman85, sheetal.pathak...
    • nod_ committed c1401cc6 on 11.0.x
      Issue #3463111 by sanket.tale, jaydeep_patel, yujiman85, sheetal.pathak...
    • nod_ committed bec7d011 on 11.x
      Issue #3463111 by sanket.tale, jaydeep_patel, yujiman85, sheetal.pathak...
  • 🇫🇷France nod_ Lille

    Welcome to all the new folks!

    Committed bec7d01 and pushed to 11.x. Thanks!
    Committed c1401cc and pushed to 11.0.x. Thanks!
    Committed 31afe9d and pushed to 10.4.x. Thanks!
    Committed bb1fad9 and pushed to 10.3.x. Thanks!

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

Production build 0.71.5 2024