🇮🇳India @DishaKatariya

Account created on 6 July 2022, almost 2 years ago
#

Recent comments

🇮🇳India DishaKatariya

I have tried to reproduce this issue but still i am getting the same results.
Attaching recordings.
Test Results are:
Click on the pagination displayed at the bottom of the page
It will show the results and take you to the top of the page

🇮🇳India DishaKatariya

Hi, I have verified this issue with on Drupal 10 with Gin 8.x-3.x-dev theme version (PHP V - 8.2.18). This is fixed now and working as expected.
Attaching the before and After recordings.
Can be move to RTBC.

🇮🇳India DishaKatariya

Hi, I have verified and tested the drupal logo issue in our 11.x dev branch. It is working fine now.

Testing steps:
- Using Drupal 11 dev line, install standard profile.
- enable Navigation module.
- goto 'Navigation settings' config page (admin/config/user-interface/navigation/settings)
- select 'custom logo'
- attach image to 'choose custom logo'
-save config
- notice: Navigation Logo is Still the 'default drupal logo'

Testing Results:
Drupal logo should be changing now when custom logo is uploaded.

Attached Before and After Screenshots as well.

Hence moving this issue to RTBC.

🇮🇳India DishaKatariya

Hi, Can someone please add the propr screenshots of this issue? so that i can be able to reproduce it properly.
Followed the given steps but changes are already looking fine to me as expected.
Thanks.

🇮🇳India DishaKatariya

Hi, I have verified and tested MR !1, with the Pets CLinic theme. The twitter logo has been updated adn working properly.
Adding the before and after screenshots below.
Can be move to RTBC
RTBC++

🇮🇳India DishaKatariya

Hi, I ahve verified and tested MR !17 with the Firma theme. The twitter logo is updated and looking fine to me.
Working as expected as well.
Attaching the before and after screenshots.
This can be move to RTBC.
RTBC++

🇮🇳India DishaKatariya

Hi, I've tried to reproduce the issue by following the aforementioned steps on a fresh Drupal 10.0dev installation with Claro,Gin,Seven,Olivero, etc. set as default theme, but the twitter icon seems to be fine and properly updated to 'X'.
Attaching the screenshots below.
can be move to RTBC
RTBC++

🇮🇳India DishaKatariya

Hi, Verified an tested MR 22, patch applied cleanly with the Festeva Live theme and the logo of the twitter in the bottom is changed.
Attaching the screenshot for reference.
Can be move to RTBC
RTBC++

🇮🇳India DishaKatariya

Hi, I have verified and tested this issue in the Drupal 10.xdev version and Gin theme version 8.x-3.0-rc9. But unable to reproduce it.
Can i get the brief steps for this issue?
The steps i have followed are:
1. Install the Drupal 10.xdev version
2. Install Gin theme and set it as by default.
3. Add some content
4. Go to Content page and scroll until the sticky header appears.

The results are:
The header is already sticky even before applying the patch.
Attaching the screenshot:

🇮🇳India DishaKatariya

Hi, I have tested this issue. and its showing the same results.Followed the same steps as given above
Steps to reproduce
Take the date picker from the example "Style guide"

In a browser on a windows device

1. Click on the date field... the date picker pops up.
2. Click on the right scrollbar of the window... the date picker disappears.
3. Click again in the date field... the date picker does not popup again.

You must first click on another field and then on the date field to popup the date picker again.
Attaching the recording below:

🇮🇳India DishaKatariya

Hi @alok_singh, I have Verified and tested MR 13. It lloks good in mobile view but Ipad devices such Ipdad mini and Ipdad Air are having the issues. It seems address is breaking there. Also breaking in the is not aligned and on 360px view.
Adding the screenshots below.
Hence moving it to needs work again.
Thanks

🇮🇳India DishaKatariya

Hi
I've tested MR !17 on Drupal version 10
MR applied successfully...

Test Result:
The validation message is displayed on the Phone no. input field.

Added Before and After screenshot for references

Can be move to RTBC
RTBC+1

🇮🇳India DishaKatariya

Hi, I have verified and tested this issue. It looks good to me.
Testing Steps:
1. Install the Drupal 10.x version
2. Set Olivero theme as default
3. Install Media library modules.
4. Now, go to the structure > add content type- article and Manage fields - add field- Field type as Media
5. Then Go to Manage Form Display- admin/structure/types/manage/article/form-display
6. Click to the Settings icon on the created Media Field and enable the show edit button.
7. then, go to the content- add content > article > Add Media and check if the edit link is visible

Testing Results:
The Edit link is Visible on the Add Media with Olivero theme.
Adding the Screenshots below:
Hence, Can be move to RTBC.
RTBC+1
Thanks!

🇮🇳India DishaKatariya

Hi, @Neha-Verma I have tried to reproduce this issue but not able to do this. I can't see the edit link even after applying the patch the results were same. Can you please provide the detail steps to reproduce this issue.
The steps i have followed:
1. Install the Drupal 10.x version
2. Install Media library modules.
3. Now, go to the structure > add content type- article and add field.
4. then, go to the content > add content > article > Add Media and check if the edit link is visible

Thanks.!

🇮🇳India DishaKatariya

Hi, I am not able to reproducethis issue. I can see only one login form even without applying the patch.
I have checked on Drupal 10.1 and 10.2 versions.
Followed the below steps:
Install the Drupal version 10.1 and 10.2
Install the theme
Visited the login page > admin login > logout
I can see only one form there.
Please correct me if i am missing anything and provide the details testing steps.

🇮🇳India DishaKatariya

Hi @adarshv I have tried by verifying the merge request !11. But the below error i have received while applying the patch.
Error: Skipped patch 'drupal8_parallax_theme.theme'.

🇮🇳India DishaKatariya

Hi Verified and tested the Merge request !177 on the Drupal 11.x for standard and Umami. This is working as expected now.
Attaching the recording below.
RTBC+1
Keeping it in review as per above comment to review the code.

🇮🇳India DishaKatariya

Hi, I have verified and tested #2 patch. But it is showing below error for me while applying the patch.
error: (Skipped patch 'moksu.theme'.)

🇮🇳India DishaKatariya

HI Verified and tested as per #11 comment and Merge request !177. It looks fine to me now.
1st item label is coming properly now and fit to the screen.
attached the recording below.
Hence moving this to RTBC
RTBC++

🇮🇳India DishaKatariya

Hi, Verified and tested Merge request !177 on the Drupal 11.x version. Patch applied successfully and working fine.
Testing steps:
1. Install the Drupal 11.x version
2. Install the navigation module
3. Reduce the browser height to a size where the Configuration menu is shown, and the People is hidden below the navigation footer.
4. Navigate to admin/content
5. Click on Configuration
6. The sub-menu children of Configuration will open below the footer, when it should actually scroll the show the sub-menu.
7. You can see the correct behavior by:
8. Navigate to /admin/structure/block
9. Resize the browser to a height that hides People
10. Click on Configuration: it will scroll to fit the sub-menu
Testing Results
When the parent menu is expanded, the navigation bar should be automatically scrolling to ensure visibility of the child menus.
Attaching the recording below.
Can be move to RTBC.
RTBC++

🇮🇳India DishaKatariya

Hi, Verified and tested #3 patch file on the D10.x version. The patch was applied successfully and looks good.
Testing steps
1. Install the D10.x vesrion
2. Install the Freelancer Zymphonies theme
3. Apply the patch and check the Twitter logo should be updated.
Testing Results
Twitter logo changed to the X
Attaching the screenshot for reference:
Can be move to RTBC
RTBC++

🇮🇳India DishaKatariya

Hi, Verified and tested the Merge request !6432 on 11.x-dev version. Its working as expected now.
Testing Steps:
1. Install the 11.x-dev version
2. Go to the Extend and enable the Languages module
3. Enable the translation module.
3. Now go to the /admin/config/regional/language/add and add any language.
3. then, Go to '/admin/config/system/site-information/translate' and try to set a longer slogan.
4. Now you will be able to see the error for the slogan added 255 characters long. and not able to add the translation.
5. Now, Apply the patch.
6. and Check if the maxlength is added and can be able to save translation slogan.
Testing Results-
Increased site slogan maxlength for the translations.
Attaching screenshots for the reference.
Can be move to RTBC.
Thanks!

🇮🇳India DishaKatariya

Hi, Verified and tested latest MR !74 on the D10 version with Barrio Bootstrap 5 Theme.
Testing Steps-
Install Drupal vesion 10.
Install the Barrio Bootstrap 5 Theme- 5.5.x version.
Go to the Appearance and set it as Base theme and Sub- theme.
Now go to the settings of the subtheme- Components > Buttons > Default button type set as Primary and Save.
Now check to the input button search should het highlighted with color near search bar.
Attaching screenshot for reference.
Can be move to RTBC.

🇮🇳India DishaKatariya

Hi,Verified and tested Issue with this Branch bootstrap_barrio-3384625 on the D10.X version with the Barrio Bootstrap 5 Theme
Version: 5.5.x-dev.
Testing steps-
1. Install the D10.x version
2. Install the theme 5.5.x version
3. Now Apply the patch and check for the hidden labels in the menu H2 tag has been removed by inspecting the element.
Testing Result
For the Hidden labels in menu an H2 tag is removed.
Attaching screenshot for reference.
Can be move to RTBC.
Thanks!

🇮🇳India DishaKatariya

Verified and tested the MR6! Getting the below error:In CurlDownloader.php line 623:

The "https://repo.packagist.org/p2/drupal/travel_zymphonies_theme.json" file could not be downloaded (HTTP/2 502 )

require [--dev] [--dry-run] [--prefer-source] [--prefer-dist] [--prefer-install PREFER-INSTALL] [--fixed] [--no-suggest] [--no-progress] [--no-update] [--no-install] [--no-audit] [--audit-format AUDIT-FORMAT] [--update-no-dev] [-w|--update-with-dependencies] [-W|--update-with-all-dependencies] [--with-dependencies] [--with-all-dependencies] [--ignore-platform-req IGNORE-PLATFORM-REQ] [--ignore-platform-reqs] [--prefer-stable] [--prefer-lowest] [--sort-packages] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--apcu-autoloader-prefix APCU-AUTOLOADER-PREFIX] [--] [
...]

composer [require drupal/travel_zymphonies_theme:^3.0] failed, composer command failed: exit status 255. stderr=

Hence not able to reviw this issue.
Thanks

🇮🇳India DishaKatariya

Adding some more screenshots for before patch of Safari browser for reference.This is fixed now.

🇮🇳India DishaKatariya

Hi,
I verified and tested the MR 6103 on the Drupal 11. x version.
Patch applied successfully
Followed the below Testing steps:
Install the Drupal version 11. x

  1. Go to Structure > Display Modes > View Modes
  2. Click Add View mode and the modal will open
  3. Under content click on Add view mode for Content
  4. Start typing the name for the view mode (small and long text)
  5. See how only the closing bracket is in the new row(not only the closing bracket appearing in the new row but the machine name also appearing in new row)
  6. If you continue with typing you will see how the actual machine name will go into the next row but the label Machine name: will stay up.
  7. Apply Patch, and check for the machine name

Test Result:
The machine name is displayed in suffixes and not broken into multiple lines
According to me this issue is no more exist. Verified this on the multiple resolutions (responsive)and browser(Chrome,firefox,safari) as well.
Attaching the screenshots for referrence below.
Can be move to RTBC
Thanks!

🇮🇳India DishaKatariya

Hi, I have verified and tested MR- merge_requests/6306 file on the 11.x version with.It is working as expected now.
Testing steps-
Install the Drupal 11.x version
Now go to the /structure/taxonomy and click on list terms
Now click to Add term
Add themultiple terms, set as one parent term and in that drag the other as child terms
Now click to save
and then click to the Delete operation
and now see on the popup which displays the parent term name with the child name as well with the notification.
Testing Result:
Orphan taxonomy terms are deleted with the notification.
Attaching screenshot for reference.
Thanks!

🇮🇳India DishaKatariya

Hi, have gone through issue mentioned and followed the below steps. I'm not able to reproduce the issue. So i didn't try with patch.
Steps i followed:
Install Drupal 9.5.10 and 10.x
Enabled the Drupal GIN theme 8.x-3.x.
Entity Browser instance with image selection.
Opened the entity browser and observed the image thumbnails. It's looks fine as expected.
Keeping it in needs review for more reviews.
Attaching screenshot for reference.
Thanks!

🇮🇳India DishaKatariya

Hi, I have verified the Current changes (#8)3390914-grid-style-field-type Branch MR on 11.x version.This is working as expected.

Followed the below testing steps:
1.Install the Drupal version 11.x
2. Fetch the latest branch
3. Now Go to the /admin/structure
4. Select content types /admin/structure/types
5. Now select any Article or Basic Page
6. Go to the Manage Fields
7. Click on Create New field button and the Grid-style field types should appear.
8. Now check with clicking page up-down keys should work as up-down keyboard navigation for grid-style radio buttons field types.
Testing Results:
With the page up and down arrow keys the grid-style radios buttons are having proper keyboad navigation as up-down now.
Hence moving this issue to RTBC.
Thanks!

🇮🇳India DishaKatariya

Hi, Verified and tested #5 patch file on 10.x version with project browser module. It is working as expected now.
Followed the below testing steps:
1. Install the Drupal 10.x version
2. Install the Project Browser module
3. Now go to the extend- admin/modules/browse
4. Observe the changes here
Attaching the screenshot for reference
Hence moving it to RTBC.
Thanks!

🇮🇳India DishaKatariya

Hi, Verified and tested #20 by taking pull from the latest branch on 11.x version. The changes were made are looking fine to me.
Followed the below steps:
1. Log in and go to /admin/config/development/configuration/single/export
2. Under Configuration type select Block.
3. Open the Configuration name dropdown
4. See the order is based on label, not machine name.
Attaching screenshot for reference.
Thanks

🇮🇳India DishaKatariya

Hi, Verified and tested the current changes on Branch 11.x version.
Followed the below testing steps:
1. Install the Drupal version 11.x
2. Fetch current branch
3. Create content type as landing page
4. Create a new field
5. select any field
6. Observe the changes on next page
Taking reference from #26 comment,
Usability i have observed here:
1. In this form there is no option available to select/ click the field, user might get confuse here: admin/structure/types/manage/test_landing_page/fields/add-field
2. Its fine to make two step form because it look more specified but to select any field the radio button option should be there.
Attaching screenshot for reference:
keeping it to Needs review state.
Thanks!

🇮🇳India DishaKatariya

Verified and tested after taking pull from this branch- 3198236-add-another-buttons. (#38) on the Drupal version 11.x. Its working as expected for the various themes.
Followed the below Testing Steps:
1. Install paragraph module
2. Create multiple paragraph types
3. Create a content type and add paragraph as reference field with 'Add mode' as buttons
4. Save the changes
5. Goto create content and check 'Add paragraphs' buttons have vertical margin
Attaching the screenshot for reference with various themes.
Hence moving it to RTBC
Thanks!

🇮🇳India DishaKatariya

Hi, I have verified this latest MR with the Drupal 11.0-dev and the Claro - issue with icon sizes is fixed now.
Followed the below steps:
1. Install the Drupal version 11.x-dev
2. Go to Appearance and set the Claro theme as by default.
3. Applied the patch with changing the font size 24px or 34 px.
4. Observe that the icon size remain fix to the size.
Attaching screenshot for reference. Moving it to RTBC.
Thanks

🇮🇳India DishaKatariya

Hi, I have verified this issue on the Drupal 9.5.11 version and Stable contrib Theme set as by default. And the Tabledrag is not broken in layout builder sidebar.
Followed the below steps:
1. Install Drupal 9.5.x-dev
2. Install stable contrib theme
3. Enable Layout builder and Layout builder modules
4. Enable Layout builder on the article content type
5. Go to Manage display
6. Select a block and click 'Move'
7. See that the drag-and-drop icon is not visible and you are unable to move the items
Attaching the screenshots before and after patch.
Moving to RTBC,
Thanks.

🇮🇳India DishaKatariya

Hi, I have verified this issue on the Drupal 9.5.11 version and Adminimal - Responsive Administration Theme set as by default. And the long labelled content types or categories are not showing the unnecessary arrow.
Followed the below steps:
1. Installed the Adminimal - Responsive Administration Theme
2. Set it as by default
3. Create a Large labelled Content type
4. Now go to views > create view> create a page and save.
5. Now select the Format as Fields and Apply
6.Now go to filter criteria and check the checkbox> Expose this filter to visitors, to allow them to change it and apply
7. Then scroll down and select Allow multiple sections and apply and Click to Save.
8. Now click to the path and check if extra unnecessary arrow isn't appearing for large content types.
Attaching the screenshots before and after patch.
Moving to RTBC,
Thanks.

🇮🇳India DishaKatariya

Hi, I have verified this issue on the Drupal 9.5.x and Drupal 1.0.1 version. And the admin toolbar isn't breaking now.
Followed the below steps:
1. Installed the Festeva Live theme
2. Set it as by default
3. Now, istalled the admin toolbar module.
4. Visited the Home icon of drupal
5. Check on hover the admin toolbar is coming fine.
Attaching the screenshots before and after patch.
Moving to RTBC,
Thanks.

Production build 0.69.0 2024