🇮🇳India @TanujJain-TJ

Account created on 7 February 2022, almost 3 years ago
#

Recent comments

🇮🇳India TanujJain-TJ

please let me know when we might expect an official Drupal 11 release for this module?

🇮🇳India TanujJain-TJ

Thanks for working on this, it is an essential module for my D11 project so Any updates or estimated D11 release dates would be greatly appreciated!

🇮🇳India TanujJain-TJ

please let me know when we might expect an official Drupal 11 release for this module?

🇮🇳India TanujJain-TJ

Just checking on when the next release for Drupal 11 support for this contrib module will be. I’m looking to upgrade site to Drupal 11, and this module is essential for the project. Any updates or estimated release dates would be greatly appreciated!

🇮🇳India TanujJain-TJ

Tanuj. made their first commit to this issue’s fork.

🇮🇳India TanujJain-TJ

Created new branch and MR for second solution also updated Issue summary. Please review.

🇮🇳India TanujJain-TJ

Tanuj. changed the visibility of the branch 3422758-no-focus-on to hidden.

🇮🇳India TanujJain-TJ

Tanuj. made their first commit to this issue’s fork.

🇮🇳India TanujJain-TJ

Removed cursor pointer for form label, it was not related to only blocks but this was throughout Drupal in editing node form or having exposed form in views. created a MR for this please review.

🇮🇳India TanujJain-TJ

Tested and verified this MR and it fixes the issue, this happens with both private and public video because of enabling this Enable Display field option in Video field setting as stated by @sijumpk. attaching screenshots for reference. RTBC++

🇮🇳India TanujJain-TJ

hey sure @smustgrave will keep that in mind. thanks

🇮🇳India TanujJain-TJ

That's what the patch does it removes the extra X icon from chrome as it is generated by default for input[type="search"] field in chromium based browsers but this extra X is not present in Firefox browser.

--- a/css/module_filter.modules_tabs.css
+++ b/css/module_filter.modules_tabs.css
@@ -109,6 +109,12 @@
 .modules-tabs__pane input.table-filter-text {
   width: 100%;
 }
+.modules-tabs__pane input[type="search"]::-webkit-search-decoration,
+.modules-tabs__pane input[type="search"]::-webkit-search-cancel-button,
+.modules-tabs__pane input[type="search"]::-webkit-search-results-button,
+.modules-tabs__pane input[type="search"]::-webkit-search-results-decoration {
+  -webkit-appearance:none;
+}

This code present in the patch/MR removes that extra X from chromium based browsers, also this extra X icon present by default doesn't update the search results and simply clears the text from field.

🇮🇳India TanujJain-TJ

Tested and verified this issue on drupal 10.2 and patch #2 works fine and fixes the issue of 'X' icon alignment in search bar and also removes the additional 'X' icon which shows up on chrome and safari browser as stated in #4. Attaching before and after screenshots for Firefox and chrome browser. RTBC++

🇮🇳India TanujJain-TJ

It seems to me that this isn't an issue for this module, but rather a bug in the core CKEditor5, and it could be linked to this. - https://www.drupal.org/project/drupal/issues/3405440 🐛 Ckeditor 5 is wrapping html tags inside p tag Active . CKeditor5 wraps all elements inside <p></p> which is breaking the UI.

🇮🇳India TanujJain-TJ

The bug was reproducible on drupal 10.1 and 11. Tested and reviewed this MR manually which fixes the focus issue for current_password field. Attaching before and after screenshots for better clarity of this issue. changing the status to NR again.

Before

After

🇮🇳India TanujJain-TJ

hey @quicksketch tried a different approach to handle this, instead of changing the warning message i tried to improve the UI for toolbar with more focus on buttons so it doesn't feel like a footer. this will also tackle the situation where if the layout has more elements the message stays on top of page and the user just scrolls past it this new button design will be visible clearly to user in such case.
attaching a patch for this, if this solutions looks good will create a MR.

🇮🇳India TanujJain-TJ

Reviewed and tested the MR. It helps improve the UX and makes it easier for users to navigate through the layout builder. attaching before and after gif to show the improvements in UI.

RTBC++

🇮🇳India TanujJain-TJ

Did a little digging related to this blank option in select list, turns out it was put in intentionally. Found two issues related to this Do not add a section when editing an empty layout, or differentiate between new layouts and existing empty layouts & Rename SectionStorageTrait to SectionListTrait
so idk if this is good idea to remove the blank select option and what complications or bugs it might cause if we remove it. might need some inputs and review of others before pushing a merge request or patch so moving it to needs review.
also this issue is not related to claro theme but layout builder module or field layout module hence updating it.

🇮🇳India TanujJain-TJ

Tested the merge request and it solves the spacing issue, attaching before and after screenshots for this. RTBC++

🇮🇳India TanujJain-TJ

Hey @Nitin-shrivastava, I am unable to reproduce this issue because I was unable to locate the layout settings option on 'Manage Form Display' for any content type, maybe it persists within 'Manage Display' tab which has 'Manage Layout' button. I have tried it with Claro, Olivero and some contributed themes but the outcome is same, this issue maybe is related to some specific contrib theme and not core issue. Please update steps to reproduce.

🇮🇳India TanujJain-TJ

Filters and search are still missing from dashboard, updating the version to 2.x.
Adding a patch file that adds Date Range Filter and Keyword Search field to search for Title, Email address & Body from Mail.

🇮🇳India TanujJain-TJ

Tanuj. made their first commit to this issue’s fork.

🇮🇳India TanujJain-TJ

Tanuj. made their first commit to this issue’s fork.

🇮🇳India TanujJain-TJ

@djsagar i tested this on drupal 10.1 and 9.5 with gin version 8.x-3.x. i was able to reproduce it.

🇮🇳India TanujJain-TJ

I was able to reproduce this on drupal 10.1. tested and verified the MR and it solves the issue, adding before and after screenshots.
RTBC ++

🇮🇳India TanujJain-TJ

I was able to reproduce this and tested and reviewed the MR on drupal 10.1, it fixed the spacing issue on dropdown menu. attaching before and after screenshots of it.

RTBC

🇮🇳India TanujJain-TJ

I tried to reproduce this bug with french, spanish and hindi language translation on drupal 10.1 and was able to reproduce it, message text was not translating and still showing in default language. after testing @boddy’s MR it didn’t solve the problem for me instead the message texts disappeared, tested with multiple languages but the issue remained same.
adding before and after MR screenshots for this and moving the status to needs work again.

🇮🇳India TanujJain-TJ

I was able to reproduce this on my local and tested the MR, it fixes the issue with unwanted horizontal scroll bar on table with no header, also MR provided in this issue 🐛 gin-table-scroll-wrapper prevents nested paragraph buttons from being clickable Fixed doesn't fix the above mentioned problem for me as well on drupal 10.1 and 9.5.
So adding before and after screenshots after testing the MR provided by @Peacog as it solves the issue.

RTBC+1

🇮🇳India TanujJain-TJ

Was able to reproduce this issue on my local, Reviewed and tested the patch with drupal 10.1 and it solved the above mentioned issue for status message with RTL language. Attaching before and after images for it.

RTBC +1

🇮🇳India TanujJain-TJ

Noticed there were some grammatical errors in exception messages such as duplicate words in the commit added, so adding a patch for it along with new tests for the function and exceptions messages not sure if it is correct, please review.

🇮🇳India TanujJain-TJ

Verified and tested patch #2, looks good to me as it removed the unused 'claro/drupal.responsive-detail' library. RTBC +1

🇮🇳India TanujJain-TJ

Tested and verified the patch it applied successfully and fixed all phpcs errors.

🇮🇳India TanujJain-TJ

Tested and verified patch, it applied successfully and fixed all the phpcs error, but the patch adds a whitespace error in readme.md file please see screenshot for reference. so updating the patch for same can be moved to RTBC.

🇮🇳India TanujJain-TJ

Verified and tested the patch and it fixes all the coding standard issues, although idk if we need fix for .js file it seems redundant to me and makes the patch quite big in size, so changing the status to needs work. if you think .js file fix is important feel free to change the status to RTBC
Thanks.

FILE: .../web/modules/contrib/ckeditor5_findandreplace/js/build/find-and-replace.js
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------------
🇮🇳India TanujJain-TJ

updated patch according to point addressed in #3, please review.

🇮🇳India TanujJain-TJ

For latest dev branch getting these below listed phpcs errors.

FILE: /datetime_range_timezone.tokens.inc
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------------
 142 | ERROR | [x] Use "elseif" in place of "else if"
 142 | ERROR | [x] Expected 1 space after closing parenthesis; found 0
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: /datetime_range_timezone.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
 1 | WARNING | Remove "project" from the info file, it will be added by
   |         | drupal.org packaging automatically
 1 | WARNING | Remove "datestamp" from the info file, it will be added by
   |         | drupal.org packaging automatically
 1 | WARNING | Remove "version" from the info file, it will be added by
   |         | drupal.org packaging automatically
 8 | WARNING | All dependencies must be prefixed with the project name, for
   |         | example "drupal:"
--------------------------------------------------------------------------------


FILE: /tests/src/Kernel/DateRangeTimezoneHelperTrait.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 36 | WARNING | [x] Inline @var declarations should use the /** */ delimiters
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: /tests/src/Kernel/TokenReplacementTest.php
--------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------------
  71 | ERROR | [x] Expected 1 blank line after function; 2 found
 111 | ERROR | [x] Expected 1 blank line after function; 0 found
 112 | ERROR | [x] The closing brace for the class must have an empty line
     |       |     before it
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: /src/Plugin/Field/FieldFormatter/DateRangeTimezone.php
--------------------------------------------------------------------------------
FOUND 4 ERRORS AND 2 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------
 40 | ERROR   | [x] Expected 1 space between double arrow and "$item"; 2 found
 40 | ERROR   | [x] Expected 1 space after "=>"; 2 found
 42 | ERROR   | [x] Expected 1 space between double arrow and "$item"; 2 found
 42 | ERROR   | [x] Expected 1 space after "=>"; 2 found
 76 | WARNING | [ ] t() calls should be avoided in classes, use
    |         |     \Drupal\Core\StringTranslation\StringTranslationTrait and
    |         |     $this->t() instead
 77 | WARNING | [ ] t() calls should be avoided in classes, use
    |         |     \Drupal\Core\StringTranslation\StringTranslationTrait and
    |         |     $this->t() instead
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: /src/Plugin/Field/FieldFormatter/DateRangeTimezoneSingleDate.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
 71 | WARNING | t() calls should be avoided in classes, use
    |         | \Drupal\Core\StringTranslation\StringTranslationTrait and
    |         | $this->t() instead
 72 | WARNING | t() calls should be avoided in classes, use
    |         | \Drupal\Core\StringTranslation\StringTranslationTrait and
    |         | $this->t() instead
--------------------------------------------------------------------------------


FILE: /src/Plugin/Field/FieldFormatter/DateRangeTimezoneFormatterBase.php
--------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------------
  8 | ERROR | [x] Missing class doc comment
 43 | ERROR | [x] Expected 1 blank line after function; 0 found
 44 | ERROR | [x] The closing brace for the class must have an empty line
    |       |     before it
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

adding a new patch to fix all these errors, please review.

🇮🇳India TanujJain-TJ

unrelated failure, retest is green. please review.

🇮🇳India TanujJain-TJ

Verified patch #2 and MR, fixed the spelling mistake issue. changes looks good to me. RTBC.

🇮🇳India TanujJain-TJ

tested and verified patch #2, it applied successfully and fixed all spelling mistakes in code comments. RTBC

🇮🇳India TanujJain-TJ

Tested and verified patch #2, patch applied successfully and fixed the spelling mistakes in commented code. patch looks good. RTBC

🇮🇳India TanujJain-TJ

updated patch after addressing points from #21, still needs tests tho.

🇮🇳India TanujJain-TJ

Tested and verified patch #51 with drupal: 10.1.x and the patch applied successfully and fixed the issue of multiple message on page. adding screenshots for reference. RTBC+1

🇮🇳India TanujJain-TJ

tested and verified patch #38 on claro and olivero themes, patch applied successfully and fixed the 'Incorrect access denied messages in branding block', attaching screenshots for reference. need RTBC+1

🇮🇳India TanujJain-TJ

Updating patch after addressing points from #25, although i am not sure about how to move forward with point 5. please review.

🇮🇳India TanujJain-TJ

Fixed CCF on #16, keeping the status to needs work as it still needs tests.

🇮🇳India TanujJain-TJ

as patch #77 does not applies

error: while searching for:
    return $update;
  });
}

error: patch failed: core/modules/system/system.post_update.php:93
error: core/modules/system/system.post_update.php: patch does not apply

adding a reroll for #77, please review.

🇮🇳India TanujJain-TJ

Adding a patch to resolve all phpcs errors.

ignored these 2 errors as it contains installation commands

FILE: /docs/enterprise_setup.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
 4 | WARNING | Line exceeds 80 characters; contains 128 characters
----------------------------------------------------------------------


FILE: /docs/freetier_setup.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
 4 | WARNING | Line exceeds 80 characters; contains 128 characters
----------------------------------------------------------------------
🇮🇳India TanujJain-TJ

Adding a patch to fix the above mentioned phpcs issues.

🇮🇳India TanujJain-TJ

Tested and verified patch #2 on drupal:10.1.x, the patch applied successfully but still getting some PHPCS errors after running phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml

need to fix below mentioned errors, chaning it to needs work again.

FILE: /entity_print/src/Asset/AssetCollector.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
 51 | WARNING | \Drupal calls should be avoided in classes, use dependency
    |         | injection instead
 52 | WARNING | The removal-version 'entity_print:3.0.0' does not match the
    |         | lower-case machine-name standard: drupal:n.n.n or
    |         | project:n.x-n.n
--------------------------------------------------------------------------------


FILE: entity_print/src/Asset/AssetRenderer.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 71 | WARNING | Variable $build inside unset call is undefined.
--------------------------------------------------------------------------
🇮🇳India TanujJain-TJ

@dpi sorry for that null check i interpreted it wrong at first updating the patch to correct all the changes required, also these out of scope changes are just PHPCS fixes i got while running this, did it so the patch doesn't fail Custom commands check. adding new interdiff with #32.
phpcs --standard="Drupal,DrupalPractice" --extensions="php,module,inc,install,test,profile,theme,css,info,txt,md,yml"

🇮🇳India TanujJain-TJ

Adding a new patch addressing points from #34 and #35, and also fixing some phpcs errors. please review.

🇮🇳India TanujJain-TJ

As patch #15 doesn't apply and throws error,

error: patch failed: src/CognitoToken.php:184
error: src/CognitoToken.php: patch does not apply
error: patch failed: src/Form/Email/UserLoginForm.php:192
error: src/Form/Email/UserLoginForm.php: patch does not apply
error: patch failed: src/Plugin/rest/resource/CognitoUserRegistrationResource.php:200
error: src/Plugin/rest/resource/CognitoUserRegistrationResource.php: patch does not apply
error: patch failed: tests/src/Kernel/Email/LoginFormTest.php:20
error: tests/src/Kernel/Email/LoginFormTest.php: patch does not apply

adding a new patch to remove phpcs errors and addressing points from #12, #14, #17, please review.

🇮🇳India TanujJain-TJ

patch #25 doesn't apply on drupal 10.1.x and throws this error

Checking patch core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php...
error: while searching for:
    // url.

    // First, check if processing conditions are met.
    if (!($request && !empty($options['route']) && $this->hasLowerLanguageNegotiationWeight() && $this->meetsContentEntityRoutesCondition($options['route'], $request))) {
      return $path;
    }


error: patch failed: core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php:105
error: core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php: patch does not apply

adding a new patch to fix CCF on #28, adding interdiff and reroll diff. please review

🇮🇳India TanujJain-TJ

Adding a new patch addressing points from #11, please review.

🇮🇳India TanujJain-TJ

after applying #8 got this phpcs error

FILE: /src/Form/EntitySchemaConfigurationForm.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------
 272 | ERROR | [x] Array indentation error, expected 8 spaces but found 10
 273 | ERROR | [x] Array indentation error, expected 8 spaces but found 10
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: src/Plugin/DataType/SchemaOverrides.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 41 | WARNING | Unused variable $override.
--------------------------------------------------------------------------------

Adding a new patch to fix this error and for now commenting the code to remove unused variable error as it was todo, also addressing changes mentioned in #14.

🇮🇳India TanujJain-TJ

adding new patch addressing points from #4.

Production build 0.71.5 2024