Created MR for the issue. Please review.
Thanks
Attached screenshot of the same
Sonal Gyanani → created an issue.
MR!5 has been successfully applied to the Drupal 9.5.x-dev, and it is functioning as expected.
Changes and Outcomes:
As a result of this merge:
The UI has been updated to remove the icon when no values are present in either the phone number or email fields.
Attached screenshot for reference.
Thanks
Applied MR!1 on Drupal 9.5.x it is working fine, preview icon is vertically center-aligned with content.
Attached screenshot for reference.
Thanks
Hi @dpacassi,
I am not able to reproduce the issue can you brief the steps.
I checked on Drupal 9.5.x and Drupal 10.0.x its working fine without showing any error.
Thanks
I applied MR!16 to Drupal 9.5.x, and the issue has been resolved. Bullets and numbers have been removed.
I have attached screenshots for reference.
Thank you.
Applied MR!5 on Drupal 9.5.x, and the issue got fixed.
Attach screenshot for reference
Applied MR!2 its working fine only I changed the background color of a button as the white background of button is not visible on the white background of site.
Attached screenshot for reference.
Please review.
Thanks
Sonal Gyanani → made their first commit to this issue’s fork.
Verified MR!3 now the content is visible on the front page, and the issue is fixed.
Adding screenshots for reference.
Thanks
Hi @mike vogel, where is the solution for this issue you changed its status from active to needs review. I think you forgot to add a solution to this so change the status to needs work.
I am also not able to reproduce the issue can you please brief on this?
Thanks
Sonal Gyanani → made their first commit to this issue’s fork.
Applied and verified MR!5 its working fine now blocks placed in the footer region are rendered at front end.
Attached screenshot for reference and moved it to RTBC.
Thanks
Applied patch#2 it is working fine but the asterisk mark looks stretched we can fix it by providing the same height and width
.form-required:after {
background-image: url(data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHdpZHRoPSIxNiIgaGVpZ2h0PSIxNiI+PHBhdGggZmlsbD0iI0VFMDAwMCIgIGQ9Ik0wLDcuNTYybDEuMTE0LTMuNDM4YzIuNTY1LDAuOTA2LDQuNDMsMS42ODgsNS41OSwyLjM1QzYuMzk4LDMuNTUzLDYuMjM3LDEuNTQ0LDYuMjIsMC40NDdoMy41MTEgYy0wLjA1LDEuNTk3LTAuMjM0LDMuNi0wLjU1OCw2LjAwM2MxLjY2NC0wLjgzOCwzLjU2Ni0xLjYxMyw1LjcxNC0yLjMyNUwxNiw3LjU2MmMtMi4wNSwwLjY3OC00LjA2LDEuMTMxLTYuMDI4LDEuMzU2IGMwLjk4NCwwLjg1NiwyLjM3MiwyLjM4MSw0LjE2Niw0LjU3NWwtMi45MDYsMi4wNTljLTAuOTM1LTEuMjc0LTIuMDQxLTMuMDA5LTMuMzE2LTUuMjA2Yy0xLjE5NCwyLjI3NS0yLjI0NCw0LjAxMy0zLjE0Nyw1LjIwNiBsLTIuODU2LTIuMDU5YzEuODcyLTIuMzA3LDMuMjExLTMuODMyLDQuMDE3LTQuNTc1QzMuODQ5LDguNTE2LDEuODcyLDguMDYyLDAsNy41NjIiLz48L3N2Zz4K);
background-size: 8px 8px;
content: "";
display: inline-block;
vertical-align: super;
line-height: 1;
height: 8px;
width: 8px;
}
Thanks
Applied and verified patch#3 twitter icon changed with the new icon.
Attached screenshot for the reference.
RTBC+1
Thanks
Applied and verified MR!2
Results:-
core_version_requirement key is added and now the theme can be installed on Drupal versions greater than 9.
RTBC+1
Applied and verified MR!9 and have made some changes to classes that are already defined changed CSS there only and add style to active pagination. Attached here screenshot for reference.
Thanks
Sonal Gyanani → made their first commit to this issue’s fork.
Hi @alok_singh
Applied MR!8 it works fine when the box is closed but when we open it's position gets changed and is not properly center aligned.
So need to work it on again.
Attached screenshot for reference.
Hi alok_singh,
MR!11 working fine but it adds additional space because of width can we change it from 35px to 15px and remove margin-left from description.
input[type="radio"],
input[type="checkbox"] {
width: 15px !important;
}
.form-type-radio .description, .form-type-checkbox .description{
margin-left:0px !important;
}
Attached screenshot for reference.
Thanks
Applied and verified MR!3 on Drupal 9.5.x. Now duplicate keys are removed from libraries.yml file.
Can be move to RTBC+1
Thanks
Applied MR!2 on Drupal 9.5.x and it is working fine error resolved.
For now, it is commented we can remove it from the code.
Thanks
Applied MR!8 on Drupal 9.5.x and it is working fine.
Results:-
Added space between read more button and tags
I think we can update padding-left to padding only in below code
ul.inline li {
padding-left: 0;
}
ul.inline li {
padding: 0;
}
Attached here screenshots for reference
Thanks
Hi @alok_singh can you please brief about the issue do you want space for the whole content region or a specific section because the solution you provided will wrap the whole content region in a container if I want some section at front-page in full width and some in the container then this will wrap whole in the container.
For this, we can create different templates for that particular node, block or view, and wrap that in the container.
Please suggest.
Thanks
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Created MR for the issue.
Please review.
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Created MR for the issue.
Please review.
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Nigel Cunningham → credited Sonal Gyanani → .
Created MR for the issue.
Please review.
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Thanks @himanshu_jhaloya, MR looks good all errors and warnings are fixed now.
So moving it to RTBC
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Thanks @Jay Jangid, MR looks good to me.
So moving it to RTBC
Sonal Gyanani → created an issue.
Created MR against 2.x version and fix some formatting issues according to
Readme.md template →
format.
Please review
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Fixed some formatting issues according to
Readme.md template →
format.
Please review
Patch #2 applied cleanly, all errors and warnings are fixed.
RTBC +1
Created MR for the issue.
Please review.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Fixed some formatting issues according to
Readme.md template →
format.
Please review
Created MR for the issue.
Please review.
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.
Thanks
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review
Thanks
Sonal Gyanani → created an issue.
Created MR for the issue.
Please review.