- 🇺🇦Ukraine rybchynski
1. Fixed test issues -- (Updated `ImageStyleListBuilder.php`);
2. Added test case to `ImageAdminStylesTest.php`; - Status changed to Needs review
over 1 year ago 6:23am 30 April 2023 - last update
over 1 year ago Patch Failed to Apply - Status changed to Needs work
over 1 year ago 6:22pm 30 April 2023 - 🇺🇸United States smustgrave
For the new service a change record will be needed.
Also D10 is not using es6 so
+ * DO NOT EDIT THIS FILE.
+ * See the following change record for more information,
+ * https://www.drupal.org/node/2815083 →
+ * @preserve
+ */Doesn't apply.
Also patch seems to contain a ton of reformatting in the tests that seems out of scope for this issue and should be reverted.
- 🇺🇦Ukraine rybchynski
Thank you @smustgrave
I'm currently creating change record for the issue.
- 🇺🇦Ukraine rybchynski
A link to the change record: Added filter for the image styles on the image style list page →
Please review. - 🇺🇦Ukraine rybchynski
- Removed useless comment regarding ES6 from the js file.
- Reverted formatting code changes from the test file.
- 🇳🇿New Zealand danielveza Brisbane, AU
+++ b/core/modules/image/tests/src/Functional/ImageAdminStylesTest.php @@ -298,6 +298,15 @@ class ImageAdminStylesTest extends ImageFieldTestBase { + // Test if search image style form exists. + $this->drupalGet($admin_path); + $image_styles = ImageStyle::loadMultiple(); + $xpath = '//input[@placeholder="Filter by style name"]'; + if (count($image_styles) > 1) { + $this->assertSession() + ->elementExists('xpath', $xpath); + } +
It could be good to move this to a dedicated FunctionalJavascript test and test that the actual filtering works. ModuleFilterTest might be a good example to use
- last update
over 1 year ago Custom Commands Failed - 🇷🇺Russia zniki.ru
Thanks everyone, this is a great feature.
Should we move from patches to MR to be able to test changes? - First commit to issue fork.
- First commit to issue fork.
- Merge request !105433325363 - Added change event to filter image styles → (Open) created by Unnamed author
- 🇮🇳India ramprassad
ramprassad → changed the visibility of the branch drupal-3325363-11.x to hidden.
- 🇮🇳India ramprassad
I have updated the MR with additional event listener for change and fixed the Pipeline issues. Please check