- Issue created by @jaydeep_patel
- Merge request !8879Spacing issue between Checkbox label and button has been fixed. → (Closed) created by jaydeep_patel
- Issue was unassigned.
- Status changed to Needs review
5 months ago 2:29pm 22 July 2024 - 🇮🇳India jaydeep_patel Ahmedabad
Spacing issue between Checkbox label and button has been fixed.
- Status changed to Needs work
5 months ago 3:09pm 22 July 2024 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 3:40pm 26 August 2024 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 12:48pm 27 August 2024 - 🇺🇸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 2:45pm 27 August 2024 - Status changed to Needs review
4 months ago 7:33pm 27 August 2024 - 🇺🇸United States yujiman85
I moved the change into the Olivero theme and targeted the ID for that div. Needs a review.
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 reviewAdding 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
- Status changed to Needs work
4 months ago 6:08pm 31 August 2024 - 🇺🇸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 12:43pm 5 September 2024 - 🇬🇧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!
- 🇵🇭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.
- Status changed to RTBC
4 months ago 6:52am 6 September 2024 - Status changed to Needs work
4 months ago 12:46pm 6 September 2024 - 🇫🇷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. - Status changed to Needs review
3 months ago 4:39am 9 September 2024 - Status changed to Needs work
3 months ago 11:37am 9 September 2024 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.- Status changed to Needs review
3 months ago 1:47pm 9 September 2024 - Status changed to RTBC
3 months ago 6:26pm 10 September 2024 - 🇳🇿New Zealand quietone
I read the IS, comments, and the MR. There are no unanswered questions. I did not test this.
Leaving at RTBC.
- 🇬🇧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...
- 🇮🇳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. Automatically closed - issue fixed for 2 weeks with no activity.