Account created on 27 October 2023, over 1 year ago
#

Merge Requests

More

Recent comments

🇮🇳India Tirupati_Singh

Hi @anybody, I've added a new widget option named "Enable Compact Row" to enable/disable display inline fields when using the module. I've attached the screenshots of before and after fixes for reference. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Thanks @ressa and @dydave for looking into the feedback and making the quick changes. As @ressa noticed, the menu ordering for Admin Toolbar menus links was not alphabetically and this could affect user experience somehow.

I've re-applied the MR as a patch and it applied cleanly without any errors. After applying the patch, the menu links for Admin toolbar has been sorted alphabetically as shown in the attached screenshots.

Additionally, noticed one difference in the setting page title for Admin toolbar search config page. For Admin toolbar and tools config page, the page title has some inconsistency. For Admin toolbar setting config page title is 'Admin Toolbar settings', for tools it is 'Admin Toolbar Tools settings', however, for search it is 'Admin toolbar search settings'. As per my suggestion, the config page title for Admin Toolbar Search should be 'Admin Toolbar Search settings' to maintain the consistency for the modules.

I've fixed the title label for admin toolbar search settings page from "Admin toolbar search settings" to "Admin Toolbar Search settings" as mentioned previously in the comment. I think it would be better if consistency is maintained throughout the module. I'm attaching the screenshots of before and after fixes for reference. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi @dydave, I've applied the provided MR as a patch, and it applied cleanly without any errors. Previously, as mentioned in the issue summary there was no way to navigate to the "Admin toolbar tools, Admin toolbar search and Admin toolbar" settings however the module configuration can be access under the menu "Configuration > User Interface > Admin Toolbar Search, Configuration > User Interface > Admin Toolbar Tools, Configuration > User Interface > Admin Toolbar".

After applying the patch, new tabs have been enabled to access/configure Admin toolbar, toolbar tools and toolbar search under Admin Toolbar Settings, i.e., /admin/config/user-interface/admin-toolbar. This is a great feature enabling users to easily switch between these sub-module settings by just switching the tabs. I've reviewed the changes and the changes are working fine.

I've reviewed the MR changes and dropped a suggestion regarding the title changes for Admin toolbar settings. Additionally, noticed one difference in the setting page title for Admin toolbar search config page. For Admin toolbar and tools config page, the page title has some inconsistency. For Admin toolbar setting config page title is 'Admin Toolbar settings', for tools it is 'Admin Toolbar Tools settings', however, for search it is 'Admin toolbar search settings'. As per my suggestion, the config page title for Admin Toolbar Search should be 'Admin Toolbar Search settings' to maintain the consistency for the modules. I've attached a screenshot for the changes to be made for Admin toolbar search config page title.

I'm keeping the issue status to Needs review for now, although the changes are working fine. If the suggested changes can be applied then we can change status to Needs work or else it be changed to RTBC.

I'm attaching the screenshots of before and after the changes of the applied MR as a patch for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I was able to replicate the issue and found that when any height value, let's say 42px has been passed in the "block--system-branding-block.html.twig" template height variable then on the logo the height attribute is rendering as height="'42px'" instead of height="42px" due to which the image height is not getting height as per its attribute value.

I've applied the provided MR !136 as a patch and it applied cleanly without any errors. After applying the patch, the height attribute issue has been fixed and it is rendering as height="42px" which is the correct. Additionally, when no height value or height auto has been passed into height variable then the render logo "img" tag does not have height attribute. I'm attaching the screenshots of before and after patch applied issue fixes. As the MR is resolving the issue, hence moving the issue status to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've fixed the margin issue for Side-by-side paragraph when Grid option enabled. Please review the MR changes. I'm attaching the screenshots of before and after fixes of the issue for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi all, as the MR conflicts have been resolved hence I'm reviewing this issue as the status was not updated after resolving conflicts.
I've applied the provided MR as a patch and it applied cleanly without any errors. After applying the patch, the image stretch issue has been resolved and it is working fine. I'm attaching the screenshot of after fixes of the issue for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi @mlahde, I tried replicating the issue as per the mentioned steps to reproduce however the issue didn't replicate. After dragging hierarchical field group fields to Disable state and saving the changes, the changes are getting saved properly. I didn't encounter any issue where the arranged items that moved to disable state is not getting save. I tried replicate it for different content types and choosing all the formatter options that are available for Field group. I've attached a video clip for your reference and as per the attached reference I think there's no such issue either related to the Gin theme or Field group module.

Tested on:

  1. Gin: 4.0.6
  2. Field Group: 8.x-3.6
  3. Drupal: 11.0.5 and 10.3.6

Dragging items can have hierarchy and motion in 4 directions in the Manage Display page of any content type. If you for example add a field group (Fieldset) the field dragging handle/icon automatically changes from "Drag to re-order" to "Move in any direction".

The drag handler icon is also getting changed as for the nested fields we can change the field structure i.e., the field can be move in any direction as per the requirement under different field group items.

@mlahde I wasn't able to reproduce this, so I'll close this one. If that's still a problem for some feel free to reopen.

🇮🇳India Tirupati_Singh

Hi, I've rebased the MR. Attaching the screenshots of before and after fixes of the issue for reference. Please review the changes. Additionally, I've applied the MR as a patch, and it applied cleanly without any errors. After applying the patch the changes are working fine.

Thanks!

🇮🇳India Tirupati_Singh

I'm rebasing the MR as there's no response from @alok_singh.

🇮🇳India Tirupati_Singh

Hi, I've resolved the conflicts in the merge request. Attaching the screenshots of before and after fixes of the issue for reference. Please review the changes. Additionally, I've applied the MR as a patch, and it applied cleanly without any errors. After applying the patch the changes are working fine.

Thanks!

🇮🇳India Tirupati_Singh

Changing the issue status to Needs review as the pipeline is passing all tests.

🇮🇳India Tirupati_Singh

Hi @drakythe, I was able to replicate the raised issue. And when the module is enabled, then on visiting the /user/1 page getting the issue The "" entity type does not exist. in Drupal\Core\Entity\EntityTypeManager->getDefinition(). However, the issue is not only limited to the /user/1 page, i.e., the user having the admin role, it occurring for any user role having the permission to access "Navigation +: Use edit mode". I've fixed the issue, please review the MR changes. I'm attaching the screenshots of the before and after fixes for reference.

Additionally, I faced another issue while using this module. After installing the module and enabling it I got the error shown in the attached screenshot: navigation-plus_issue.png . Since it is not related to the current issue, I'm moving the issue status to Needs review.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've fixed the indentation issue in the template file. Please review the changes. I'm keeping the issue status to Needs work as the pipeline is failing.

🇮🇳India Tirupati_Singh

Hi @trackleft2, I've updated the form's new element name from "Enable Spaceless Filter" to "Remove whitespace between HTML tags". I think the new form element label will be easy for a normal user to understand. For more clarification, the user will have to go through the description provided for the element.

I've also updated a conditional check to the new element so as to show "Remove whitespace between HTML tags" only when configuring the field group for a node view page. Previously, the "Remove whitespace between HTML tags" option was available for both "Manage Form Display" and "Manage Display" although it was only intended for the node view to filter extra space when visiting the actual page content. However, it was also available for form display, which was not required. After the new changes, the "Remove whitespace between HTML tags" will only be available for "Manage Display" form when adding/updating a field group.

Since I've updated the code logic, I'm moving issue status back to Needs Review state from RTBC. I'm attaching the screenshots of the after fixes for reference. Please review the MR new changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi @shelane, could you please provide an update on the draft merge request MR !31 and let me know if there’s anything needed to move it forward?

🇮🇳India Tirupati_Singh

Hi @trackleft2, I've replicated the issue as per the comment #8 🐛 Browsers add space to output because of extra space in templates. Active and confirm that the issue persists. The rendered element is getting some extra spaces when the "HTML Elements" is selected as formatter for the field group. I've tested the issue after applying the spaceless filter, and on applying it, the additional space got removed.

As suggested by @trackleft2, it would be better to provide an option to users to enable/disable the filter as per their need. Hence, I've added a new configuration option, named Enable Spaceless Filter for the field group when the "HTML Element" option is selected. When "Enable Spaceless Filter" is checked, then the elements will be rendered through template applied with spaceless filter, and if not checked, then it will be rendered without spaceless filter applied.

I'm attaching screenshots of the before and after fixes of the issue for reference. Please review the MR changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch, and it applied cleanly without any errors. After applying the patch, when the logged-in user has no Access the Content overview page permission, then on clicking Back to Administration /admin page is opening and when the user has the permission then /admin/content page is opening.

Previously, clicking the Back to Administration opens 403 (Access Denied) when the user has no Access the Content overview page permission and opens /admin/content page when access is provided.

As the provided MR fixes the issue as per the proposed resolution and the changes are working fine, hence moving the issue status to RTBC. Attaching the screenshot and video clips for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi @rubymuse, I've fixed the rendering issue for the menu icons for which the user has no permission. Now the menu icons along the menu link will be rendered as per the logged user's role permission. I've allowed some administrative permission to the logged-in user role and now the menu icons are being shown as per the role as shown in the screenshot. I'm attaching screenshots of the before and after fixes of the issue. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi all, I've applied the provided MR as a patch and it applied cleanly without any errors. However, after applying the patch getting below error when configuring the mega menu.

The website encountered an unexpected error. Try again later.

Error: Call to undefined method Drupal\Core\Menu\MenuLinkDefault::getEntity() in Drupal\we_megamenu\WeMegaMenuBuilder::getMenuTree() (line 48 of modules/custom/we_megamenu/src/WeMegaMenuBuilder.php).

I've also tried applying the patch provided in the comment #4 Add translatable menu links to we_megamenu Active but getting below errors on applying:

Checking patch src/WeMegaMenuBuilder.php...
error: while searching for:
            }
          }
        }
      } 
    }
  }


error: patch failed: src/WeMegaMenuBuilder.php:448
error: src/WeMegaMenuBuilder.php: patch does not apply

I'm attaching the screenshots of before and after of the issue for reference. As the MR is not resolving the issue hence moving back the issue to NW.

🇮🇳India Tirupati_Singh

Hi @anybody @xpersonas, I've applied the latest changes in the MR as a patch and it applied cleanly without any error. On applying the patch, the issue mentioned in the comment #31 Possibility to set words as reject pattern Needs review has been resolved. However, when the module is installed and then the patch has been applied, then on visiting the module configuration page i.e., /admin/config/content/protected_forms getting the below error:

The website encountered an unexpected error. Try again later.

InvalidArgumentException: "0" is an invalid render array key. Value should be an array but got a string. in Drupal\Core\Render\Element::children() (line 97 of core/lib/Drupal/Core/Render/Element.php).

But after reinstalling the module with the applied patch then the above issue has not been encountered.

After applying the patch, a new field Pattern matching has been added with options Check whole word instances and Check within strings. The newly added field provides the option to match the added pattern to match the whole or match only the pattern within the words. Both the provided field options are working fine with the forms and blocking the form submission as per the selected option in the field. I've attached the video clip for your reference and screenshots of before and after fixes as well.

As getting the mentioned error after applying the patch, hence keeping the issue to Need works.

Thanks!

🇮🇳India Tirupati_Singh

Hi @megachriz @sourojeetpaul, I've reviewed the updated logo in the comment #21 📌 Create logo for Feeds Tamper for Project Browser Initiative Active and it looks good to me. The change in the tamper circle color now representing in the modification of the input and output data through the conversion on squre into circle indicating the module functionality well. As the latest proposed logo meets the requirement hence, moving the issue to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi all, I replicated the issue and the issue persists. Upon adding any pattern in the Reject patterns field, the user is unable to submit the form due the any word containing the pattern added in the reject patterns field. I also applied the provided MR as a patch and it applied cleanly without any errors. However, after applying the patch getting below issue:

The website encountered an unexpected error. Try again later.

TypeError: Drupal\protected_forms\Form\ProtectedFormsForm::__construct(): Argument #1 ($module_handler) must be of type Drupal\Core\Extension\ModuleHandlerInterface, Drupal\Core\Config\ConfigFactory given, called in /app/web/core/lib/Drupal/Core/Form/ConfigFormBase.php on line 57 in Drupal\protected_forms\Form\ProtectedFormsForm->__construct() (line 28 of modules/custom/protected_forms/src/Form/ProtectedFormsForm.php).

Now on visiting the module configuration page getting the issue and also on submitting the form on which the protected form is enabled getting the issue shown in the screenshot. The provided MR still need works hence moving it to Needs work.

Thanks!

🇮🇳India Tirupati_Singh

Hi @debrup, I've reviewed the proposed logo for the module as per the guidelines mentioned on the project browser. The logo meets the following requirements:

  1. The logo dimension is 512x512px and it does not have any animation.
  2. The proposed logo file size is within 10kb and the file format is png.

The logo image quality looks good however, the design at the top and bottom edged of the logo seems to be cutting at the edge "the square blue gear-box one at the top and bottom center". If there'll be some gap then it would be better. And also, as per the guidelines the proposed logo filename should be "logo.png" instead of Dubbot-fs8.png. Considering the above issues, I'm moving the issue status to needs work.

Thanks!

🇮🇳India Tirupati_Singh

Hi all, I've also tried replicating the issue and the issue persists while using the 2.0.3 version of the module. However, the issue has been resolved in the development version 2.0.x in the issue #3395325 🐛 Deprecated function: Creation of dynamic property (with php 8.2) Fixed and the provided MR has been merged as well. It will get resolved once the maintainer published the new release of the module. As the related issue has been merged on the dev version hence, changing the issue status to duplicate.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I replicated the issue and can confirm that the issue persist. Currently, to visit/add any custom permissions to the Custom Permissions module configuration page no menu item has been provided. It is hard for a normal user to find the module configuration. To access the module configuration, the users have to visit the Extend page and then after the module configuration can be accessed which is not a good user experience.
I've applied the provided MR as a patch and it applied cleanly without any errors. After applying the patch, a new menu item named "Custom permissions" has been added under the "People" menu allowing user to access the module configuration page to enhance the user experience. I'm attaching the screenshots of before and after fixes for reference. As the MR is resolving the mentioned issue hence, moving it to RTBC. Hope to see this fixes in the new release of the module.

Thanks!

🇮🇳India Tirupati_Singh

Hi all, I've tried replicating the issue, and it reproduced. I've applied the provided MR as a patch, and it applied cleanly without any errors. After applying the patch, the multivalued image field overlap issue got resolved successfully, along with the mentioned issues in the comment #17 🐛 Widget for multivalued file field overlap Active . Now, the multivalued image fields occupying the required space and are not blocking the usage of multivalued fields, as mentioned by @piridium.

Thanks to @mudasirweb and @sandip-poddar for resolving further designing issues while using this theme. Initially, the drag handler is center-aligned for the theme for all fields/tables, so it would be better if consistency is maintained for the theme. As changes made for the handler are not affecting the other functionality while using the theme, hence moving the issue to RTBC. I've attached the screenshots of before and after fixes for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi @andy-blum, I've added the gap between the site logo and site title & slogan. Attaching the screenshot of the after fixes for your reference. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi @shivam_tiwari, I can confirm that after applying the latest MR's changes as a patch the mentioned issues in the comment #19 has been fixed and it is working fine. I'm attaching the screenshot of after fixes for reference hence moving the issue status to RTBC. Thanks for the quick fixes.

Thanks!

🇮🇳India Tirupati_Singh

Hi all, I've applied the provided MR as a patch and it applied cleanly without any errors. However, after applying the patch the issue does not get resolved as the menu attributes are not added to the menu item. Additionally, the added attributes value are being added to link attribute instead of being added to its individual attributes. I'm attaching screenshots of the before and after fixes along with the menu item configuration for the attributes. Hence, moving the issue to NW state.

Thanks!

🇮🇳India Tirupati_Singh

Hi @rajat4u, I've applied the provided MR as a patch and it applied without any errors. After applying the patch, the new configuration has been provided to add rel attribute to the formatter for both Image and File type fields. On enabling the rel attribute for the field, the rel attribute is being added to the DOM for download link. The provided new rel attribute configuration is working fine. I'm attaching the screenshots of before and after fixes for the issues and moving it to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi @sourojeetpaul, I've updated the MR. Please review the changes.
Thanks!

🇮🇳India Tirupati_Singh

Hi @sourojeetpaul, I've added the configuration fields for social media icons. Additionally, I've added a field to Show/Hide social media title element. Please review the MR changes. Attaching the screenshots for your reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've reviewed the proposed logos for the Module as per the provided review steps.
I liked the logo jQuery-UI-Resizable.png provided in the comment #5 📌 Project Browser: Create a logo for jQuery UI Resizable Active as it designed as the proposed logo is representing the resizing icon instead of crop. The logo proposed by @AaronDeutsch seems that the module's functionality is to crop instead on resizing.

The proposed logo meets the following requirement:

  • The logo dimension is 512x512px.
  • The file format is PNG.
  • No animation has been used in the logo.
  • Better image quality.
  • The logo file size is within the 10kb.
  • Logo representing the module functionality.

As the proposed logo is meeting all the requirements hence moving it to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've resolved the pipeline issues for MR !16 only as it is resolving the issue, and it has been verified by @selfirian and @leducdubleuet. The sidebar can also be hidden by clicking outside the sidebar region. Please review the MR changes and verify whether the changes are working fine or not.

Thanks!

🇮🇳India Tirupati_Singh

Hi @raynaldmo, follow the instructions provided in the block configuration by visiting the URL /admin/structure/block to download the libraries. All the procedures have been mentioned there to use the other icon pack. I'm attaching the screenshot for your reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I'm also facing the same issue for tel link. The telephone link is also being shown as external link icon when Use Font Awesome icons instead of images config is enable. I've applied the provided MR as a patch and it applied cleanly without any errors. I can confirm that the MR is resolving the issue successfully. I'm attaching the screenshots of before and after fixes of the issue.

Thanks!

🇮🇳India Tirupati_Singh

Hi all! I've also replicated the issue following the steps to reproduce and found that the problem was caused by the use of the spaceless filter in the Twig templates. I have removed the filter from the affected templates, please review the MR changes. I'm attaching the screenshots of the before and after fixes for reference.

@doxigo, I have only removed the filter from the templates associated with the address module. Would you recommend removing the filter from all template files, or should we limit the changes to these specific ones?

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch and it applied cleanly without any errors. After applying the patch the logo bottom cut issue has been resolved successfully. However, the current logo dimension rendered is large as compared to the previous one. If the image dimension is not an issue then moving the issue to RTBC. I'm attaching the screenshots of the before and after fixes for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi @nk, I've tested the mentioned issue on 2.0.x-dev and when the number of slide is less than 2 getting a warning message as shown in the attached screenshot when saving/updating the node. And confirm that the changes are working fine, did not get any console errors when the slider has only one slide. I'm attaching the screenshot for your reference. As the changes are working fine moving the issue to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi @dhruv.mittal, I've applied the provided MR as a patch and it applied cleanly without any errors. After applying the patch, when the anonymous user tries to access the admin page getting Access Denied along with the login form as configured through the module configuration. After logging, the user is redirected to the hit URL only. I'm attaching the video clips of the before and after fixes for reference. Moving the issue status to RTBC as the MR is resolving the issue.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I was also able to replicate the issue for seven theme on Drupal 10.3.6 and confirm that the issue still persists. I've applied the provided MR !13 as a patch and it applied cleanly without any errors. After applying the patch the hover effect issue has been fixed and now there's no issue while hovering the action buttons.

I've also tried replicating the issue mentioned by @ehsa_kundu in the comment #27 🐛 In ui-dialog box at Views button hover effect is odd Active and confirm that the issue persists for the Delete action button visibility issue occurring after clicking anywhere except the Delete button. On applying the patch this issue has also been fixed.

As the provided MR !13 fixing all the issues hence, moving it to RTBC. I'm attaching the video clips of before and after fixes for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch and it applied cleanly without any errors. After applying the patch, a new configuration field 'Enable for' has been provided to Enable/Disable the module functionality for the Admin Theme or Frontend Theme. When the Admin Theme option is unchecked then the messaging shown is unaltered. The provided new field functionality is working fine. I'm attaching the screenshots of the before and after fixes for reference and moving the issue to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've fixed the external link aria-label issue. Please review the changes.
Thanks!

🇮🇳India Tirupati_Singh

Hi, I've reviewed the latest changes and now the Fediverse icon is being shown. I'm attaching the screenshot of the fixes for reference. Moving issue status to RTBC as the changes are working fine.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've fixed the Trash icon not being shown in the navigation while using the Trash module. Please review the changes. I'm attaching screenshots of before and after fixes of the issue for reference.

Thanks!

🇮🇳India Tirupati_Singh

Hello there, I've also tried replicating the issue and found that the Trash icon is displayed in the navigation. However, when using the Gin theme, the icon does not appear as mentioned by @dqd in the comment #6 🐛 Add navigation icon for Trash Active . I think the issue should be reported for Gin theme rather than the Trash module. Attaching the screenshots for reference.

Thanks!

🇮🇳India Tirupati_Singh

Moving back to Needs Review as the mentioned error in the file 3188136-nr-bot.txt has been fixed by @Bhanu951.

🇮🇳India Tirupati_Singh

Hi @redseujac @alexpott, I've fixed the site slogan issue while using the Olivero theme. Now the site slogan is being shown for all the cases when the site slogan, site name and site logo have been enable on site-branding block configuration for the theme. I'm attaching the screenshots of the fixes for the issue. Please review the MR changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi @awset, I've applied the provided wysiwyg-support-3076448-43.patch in #43 WYSIWYG support on each table cell for better content experience Needs review , it applied cleanly with no errors. After applying, each table cells have user input format option. Now user can change the input format for table cells. The changes are working fine while using the module. However, after applying the patch, export table data as CSV feature is impacted due to the new changes. Previously the table data was exported as the actual data/value but after applying the patch table data is being stored as simply 'Array' text instead of array. And when the table is being imported the table created is simply containing the text 'Array' instead of actual data due to the incorrect data present in the CSV file. Previously the import and export table data as CSV was working fine. I'm attaching the screenshots of the before and after fixes for your reference. As the module functionality is impacted due to the latest patch, hence moving back the issue status to Needs work.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch and it applied cleanly without any error. After applying the patch the view blocks pagination is now appearing as horizontal, in addition the styling has also been improved for the pagination. I'm attaching the screenshots of before and after fixes for reference. As the changes are working fine, moving it to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've fixed the alignment issue for the paragraph label while using the Modal as field widget config. I'm attaching the screenshots of the before and after fixes for reference. Please review the MR changes.

🇮🇳India Tirupati_Singh

Added comment regarding the changes made. Please review the MR.

🇮🇳India Tirupati_Singh

@manishvijay02, can we use any other icon library for fediverse icon? And I think it would be better if the font-awesome cdn can be added in a new library as the theme is adding all the cdn using the library. It should not be hard coded in the html.html.twig file.

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch and it applied cleanly with no errors. After applying patch now social media link fields for Fediverse and Mastodon under the theme configuration. Now the Mastodon social link icon is being shown in the social icons. However, the Fediverse social icon is not being shown in the social icon footer after input the url in the Fediverse field. I'm attaching the screenshots of before and after fixes for reference. As the social icon is not being shown for Fediverse hence, moving back issue to Needs work.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch and it applied cleanly without any errors. After applying the patch the configuration page of the module is now accessible and the module functionality is working as expected, tested on Drupal 11.0.5. I'm attaching the screenshots of the before and after fixes for reference.

Thanks!

🇮🇳India Tirupati_Singh

@saschaeggi, I've re-reviewed the recent changes. Everything is working fine and the MR can be merged.

🇮🇳India Tirupati_Singh

Hi @esha_kundu, I've fixed the design issue of single item list for seven. I've attached the screenshot of after fixes. Please review the changes.

I've reviewed the MR!40 and found you have included the styling for gin, claro and seven theme in the newly created file select2.single-style.scss however separate scss files are present for these theme styling. IMO we could maintain the discrete structure of styling as present in the module.

The reason why I included the styling in one file and not the existing files is because a new library is being attached with the Single item style widget when this functionality is enabled. If the change was to be kept in the existing file, the CSS would load even if the Single item style option was disabled.

🇮🇳India Tirupati_Singh

Hi @jurgenhaas, thanks for the quick fixes. I've reviewed the latest changes made for simple_sitemap and confirm that the sticky actions button is working fine while using the module on gin theme. The memory allocation issue has been resolved successfully. And the sticky actions button is working fine for node edit, views as well as simple_multistep module.

However, I didn't encounter the issue mentioned in the comment #35 🐛 Ensure sticky action buttons to work with modals and ajax refresh-calls Needs review . While using the simple_sitemap module, I'm not getting any gin more-trigger action buttons (three dots) on gin theme. I'm attaching the screenshots of before and after fixes for your reference.

As the mentioned issues have been resolved successfully and didn't get any issues while using the theme hence, moving the issue status to RTBC.

Thanks!

🇮🇳India Tirupati_Singh

Hi @sourav_paul, I've resolved the merge conflicts. Please review the MR changes.

Thanks!

🇮🇳India Tirupati_Singh

@saschaeggi, I've reviewed the latest changes for the typo fixes. Now the sticky actions button issue has been resolved successfully for node edit, views as well as for simple_multistep. The actions button is working fine when using the simple_multistep module and the node is now being save.

However, I'm also encountering the same issue Fatal error: Allowed memory size of 1073741824 bytes exhausted (tried to allocate 20480 bytes) in /app/web/core/lib/Drupal/Core/Theme/ThemeManager.php on line 204 as mentioned by @hmendes in the comment #32 🐛 Ensure sticky action buttons to work with modals and ajax refresh-calls Needs review for simple_sitemap module while using the gin theme. The issue has aroused after the typo fixes. Hence, I'm keeping the status to Needs work although the issues for node edit, views, simple_multistep has been resolved.

I'm attaching the screenshots of before and after fixes for your reference.

🇮🇳India Tirupati_Singh

Hi @deepak5423, I've fixed the eslint and cspell issues reported by gitlab-ci. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi @sourojeetpaul, updated the admin toolbar search design as per the provided feedback. Uploading the updated patch. Attaching the screenshot of the fixes for the issue for your reference.

🇮🇳India Tirupati_Singh

Hi @saschaeggi, I've tested the new changes made and after the new changes the actions button is not working for simple_multistep module. The Next button is not working for simple_multistep module, no action is being triggered on clicking the next button. Additionally, the actions button is also not being shown for node-edit form either when the Enable sticky action buttons is enabled or disabled. The recent changes has also affect other form-edit also, on enabling the Enable sticky action buttons the Save button is also not being shown on the theme setting page. Attaching the before and after screenshots for your reference.

🇮🇳India Tirupati_Singh

Hi @deepak5423, I've converted the patch into MR. Now the module is working fine on Drupal 11. Getting no issues while using the module on Drupal 11. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi @sourojeetpaul, I've fixed the admin toolbar search styling issue and updated the previous patch for the same. I'm attaching the screenshots of the before and after fixes of the issue for your reference.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've fixed the decimal value issue for the image tag width attribute while using the ratio_crop image style effect. I'm attaching the screenshots of the before and after fixes for reference. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi all! I've updated the logo as per the suggested changes. I've increased the logo text and updated the module name text case as per the module page. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've removed the xs breakpoint for bootstrap 5 and added the xl, xxl for the carousel and grid. I'm attaching screenshots of the before and after fixes for reference. Please review the changes.

Thanks!

🇮🇳India Tirupati_Singh

Hi, I've applied the provided MR as a patch and it applied cleanly with no errors. Additionally, applied the provided masquerade-drop-sessions-fix-2.x.patch patch on 2.x branch and it also applied cleanly with no errors. Before applying the patch, having more than the allowed active sessions, all real sessions are logged out. After applying the patch, only the oldest real session logged out. I'm attaching the screenshot of configuration of the session limit for reference. Moving issue status to RTBC.

🇮🇳India Tirupati_Singh

Hi all! I've reviewed the proposed logo for the module and it meets the following requirements:

  1. The logo dimensions is 512x512px.
  2. Image file format PNG
  3. No animation has been used in the logo.
  4. Image quality is fine.
  5. The image file size is 9.9kb.
  6. Provided logo is representing the module.

@sourojeetpaul, can the ball with the icon XML, JSON size be increase as these text will not be completely visible on the small size when the logo used on the project browser page? The XML text on ball is visible but the JSON text on the ball is not clearly visible in the logo on 512x512px.

🇮🇳India Tirupati_Singh

@mandclu, the css porperty .add-to-calendar { display: inline-flex;} was being used for both Modal dialog and List of links formatter while using the module and due to this css the modal was always being opened. On using the css porperty .add-to-calendar[open], span.add-to-calendar { display: inline-flex; } the modal issue resolved successfully, additionally using the open attribute the css the icons in the modal also wrap as group for the narrow spaces.

To reproduce the issue for [open]: You just have to select Modal dialog in the Links display formatter and visit the node where the date field has been used. You'll notice that the modal is being open always and the icons on the modal for narrow spaces didn't wrap properly. This issue can be resolve for modal if we also use css display:inline-flex; only when the modal opens.

And the changes in the MR !25 is working fine for the List of links for date field.

🇮🇳India Tirupati_Singh

Hi @sourojeetpaul, I've reviewed the updated logo and it meets the following requirements:

  1. Logo dimensions: 512x512px
  2. File format: png
  3. No animation has been used in the logo.
  4. Image quality is fine.
  5. The proposed logo represents the module functionality.

As the logo file size 10kb is not a hard and fast requirement hence moving the issue status to RTBC.

🇮🇳India Tirupati_Singh

Hi, I've tested the new changes made in the provided MR and confirm that the tags overflow design for mobile devices issue has been resolved successfully. Additionally, the hover effect for tags has also been implemented and its working fine. I'm attaching the after fixes screenshot for reference. Moving the issue status to RTBC as the changes are working fine.

Thanks!

Production build 0.71.5 2024