🇺🇸United States @benabaird

Account created on 15 January 2018, about 7 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States benabaird

This was added in the 1.2.2 release but never marked as fixed.

🇺🇸United States benabaird

This issue was caused by the fix in Modal/Dialog type behaviors are reversed in JavaScript 🐛 Modal/Dialog type behaviors are reversed in JavaScript Active not updating the field widget. Updated the field widget and added a test to check this.

At this point I don't think we want to add the configuration option added in https://git.drupalcode.org/issue/multiselect_dropdown-3492120/-/tree/3492120-modalbreakpoint-uses-enum, and instead always display it as a dropdown on edit forms as was the original intent.

🇺🇸United States benabaird

benabaird changed the visibility of the branch 3492120-modalbreakpoint-uses-enum to active.

🇺🇸United States benabaird

benabaird changed the visibility of the branch 3492120-modalbreakpoint-uses-enum to hidden.

🇺🇸United States benabaird

Ah I see. Sorry, I was confused by the issue description and the mention of Select2.

🇺🇸United States benabaird

Thanks for using the module!

How to implement non-standard input types is documented in the readme.

🇺🇸United States benabaird

This reduces the number of tests run from 75 (with 526 assertions) to 61 (with 436 assertions).

Time to run locally reduced from 5 minutes to 3 minutes.

🇺🇸United States benabaird

Failed PHPStan test is not a part of this work, tracked in Fix next minor PHPStan issue in tests 📌 Fix next minor PHPStan issue in tests Active .

🇺🇸United States benabaird

Very interesting, I was not aware that core considered these unique (it is documented).

Digging, the exception only is thrown when using ::create() from a bundle class.

There's some ambiguity in core on this, so closing this issue for BCA.

Thanks for looking, and for this very useful module!

🇺🇸United States benabaird

The failing phpcs tests in MR16 are in files that were not touched by this work, so I've left them as-is to be resolved in a different issue.

🇺🇸United States benabaird

Mistakenly pushed to 1.1.x. Tests pass, closing.

🇺🇸United States benabaird

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

🇺🇸United States benabaird

benabaird changed the visibility of the branch 3436536-add-attribute-support to active.

🇺🇸United States benabaird

benabaird created an issue.

Production build 0.71.5 2024