๐Ÿ‡ฎ๐Ÿ‡ณIndia @bebalachandra

Account created on 30 May 2022, over 2 years ago
  • Developer L1 Drupal FullStack at Materialย 
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

bebalachandra โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Verified the issue on 11.x-dev.

Steps to be followed to verify the issue.

Scenario 1:

1. Navigate to any content type by admin > structure > content types.
2. Edit content type by clicking manage fields.
3. Navigate to manage display tab.
4.navigate to layout option section and enable "use layout builder"
5.Save the changes and click "manage layout" button.
6.Refresh the page and you will see "unsaved changes" message.

Scenario 2:
1. Navigate to any content type by admin > structure > content types.
2. Edit content type by clicking manage fields.
3. Navigate to manage display tab.
4.navigate to layout option section and enable "use layout builder" and "Allow each content item to have its layout customized."
5.Create a node and Click the "Layout" tab.
6.Refresh the page and you will see "unsaved changes" message.

Issue was there in both scenarios before applying the MR!9333.

After The MR!9333 issue resolved for both scenarios. I suggest move this issue to RTBC

Attached screenshots for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Verified the issue in drupal 10.2.x with webform version 6.2.x-dev.

Steps followed to reproduce the issue:
1. Install drupal 10.2.x version.
2. Install webform module version 6.2.x-dev.
3. Navigate to admin > structure > webform.
4. You will find existing webform named "contact". also we can configure new webform by clicking on "add webform"
5. I have verified by editing existing webform "contact"
6. Click on the "build" button located right side of the "contact" webform.
7. Click on "add element"
8. Search for "radios" or "checkbox".
9. click on "add element" right next to the element.
10. configuration window will open, navigate to advanced tab.
11. After the patch/ MR you will see "label attributes" is visible by default.
12. Before the patch/MR "label attributes" is visible only when "Wrapper type and attributes > Wrapper type" is set to "Form element".

MR!512 and #5 patch solves the issue reported here.

Hence I suggest to move this to RTBC

Attached screen recordings for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Created MR using the patch file created from @ab.shakir. verified on local tabs block is visible in layout builder.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

bebalachandra โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Fixes verified on 11.x-dev.

I was able to reproduce the issue on my local environment. After applying fix from the MR!9340 issue resolved.

Steps followed for reviewing:
1. Installed 11.x-dev on local.
2. Navigate to admin > structure > Block layout
3. Make sure you are on claro theme tab for block placement.
4. Disable all the blocks which are placed in Header and Breadcrumb region.
5. Navigate to home page and check for the header.

After the MR!9340 header is not shown when the region is empty.

Attached screenshots for reference.

Issue mentioned is resolved after the MR. I suggest move this issue to RTBC

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

I have tried to reproduce this issue multiple times. Tried with standard and minimal both methods, unable to produce the issue mention.

I am using colima and DDEV

version: 11.x

Attached screen recording for reference. For me things are working fine.

If there is any specific scenario where this is happening, please provide more details on that.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Hi @marc.bau

"This happens also to a few other modules that create sub menus"

For visibility can you please mention those module names where you have faced similar issue?

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

As @jrockowitz mentioned there is no feature implementation for now, I have skipped MR!356 from testing (however MR failed test cases). Tested patch #4. followed all the steps which are given in "Steps to reproduce"
After overriding blockSubmit function the behaviour is same as before while uploading file and submitting. If and only we want blockSubmit function to be overridden then we can use patch #4. else we can use the module AS-IS.
Attached screenshot for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

In that case, better we can consider #5 which is mainly focused on this ticket.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Tested patch #5 and patch #6 on my local drupal 10.1. Firstly Please note that both patches are same, only the difference in patch #6 is removal of 171 line i.e " $export_options += $submission_exporter->getDefaultExportOptions();". Done testing with this line of code and without also. Both cases the result is same. "--excluded-colums" is working as expected after applying both patches. attaching Screenshot for reference. Let maintainer decide which is the best patch to consider. Both patches resolve the issue which is stated here.

We can move this to RTBC+1

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Reviewed MR!10 on drupal 10.1 on my local. this MR will resolve the issue reported here. Thanks @gisle for highlighting the MR which I have not considered while reviewing. compared both MR!10 and patch #6. patch #6 carries the same changes which are there in MR!10.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Tested Patch #6 on drupal 10.1.

As stated in description error is reproducible using the steps below

1. Go to this path : admin/config/content/node-title-validation
2. Configure the module by filling up the required inputs
3. Save the module configurations
4. You will get an error

Attached Before patch screen recording and screenshots for reference.

After applying patch #6 error resolved successfully. I am able to configure the module and messenger is coming up when clicked on save button. MessengerInterface is implemented successfully.

We can move this to RTBC+1.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

All good on 10.0. Its showing as token is upto date. I have not faced any update errors which is stated in #3. Attached ss for reference.
Thanks.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Before adding #3 patch I have observed that css/js aggregation settings are there in performance form and as mentioned in the summary since drupal 10.1 there is a development form for development settings. Twig debugging , cache settings are there in development form. I have added screenshots for both the observation i.e 1. Development form Before Patch and 2.Performance form Before Patch.

navigate to Admin>configuaration>development>Performance to view performance form
navigate to Admin>configuaration>development>development settings to view development settings form

As per the scope of this issue CSS/JS aggregation settings needs to be moved to development settings form. Installed drupal 11.x on my local and Applied #3 patch on that. As per the scope CSS/JS aggregation settings are moved to development settings form from performance form. patch is working as expected. Attached screenshots of development settings form and performance form for reference. i.e 1. Development form after Patch 2.Performance form after Patch.

Hence suggest to moving this issue to RTBC +

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

I have done these steps to replicate the issue.

Installed webform 6.2.0-beta6 on my local
Created single check box webform
Enabled webform share
In share settings I have unchecked "Display title on shared form"

I have found that before applying the #4 patch title was showing on shared webform.

I have applied #4 patch and the issue which I am facing got resolved. I have verified the shared webforms in test page of shared webforms. attaching a video and screenshots before and after applying the patch.

Hence #4 resolves the issue which is mentioned here. Moving to RTBC.

๐Ÿ‡ฎ๐Ÿ‡ณIndia bebalachandra

Hi @guardiola86, as mentioned by @Indrapatil #2 patch applied successfully and I am unable to reproduce the error which is reported. Things are working as expected. We can move this to RTBC.

Production build 0.71.5 2024