Convert ViewsExposedForm plugin discovery to attributes

Left a small comment.

sorry, wrong issue, this was meant for https://www.drupal.org/project/drupal/issues/3421010 📌 Convert ViewsArgumentValidator plugin discovery to attributes Active

el7cosmos → closed merge request !6855

Seems all feedback has been addressed

Searching for @ViewsExposedForm and all 2 instances have been replaced.

Open threads on MR.

Feedback appears to be addressed.

Left some comments on the MR

Suggestions appear to be added.

Committed and pushed 4586e9fc62 to 11.x and 5d7c6f182a to 10.3.x. Thanks!

diff --git a/core/modules/views/src/Attribute/ViewsExposedForm.php b/core/modules/views/src/Attribute/ViewsExposedForm.php
index 1fca4a2f60..6f9be79da8 100644
--- a/core/modules/views/src/Attribute/ViewsExposedForm.php
+++ b/core/modules/views/src/Attribute/ViewsExposedForm.php
@@ -28,19 +28,21 @@ class ViewsExposedForm extends Plugin {
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $short_title
    *   (optional) The short title used in the views UI.
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $help
-   *   A short help string; this is displayed in the views UI.
+   *   (optional) A short help string; this is displayed in the views UI.
    * @param string[]|null $display_types
-   *   The types of the display this plugin can be used with.
+   *   (optional) The types of the display this plugin can be used with.
    *   For example the Feed display defines the type 'feed', so only rss style
    *   and row plugins can be used in the views UI.
    * @param string[] $base
-   *   The base tables on which this exposed form plugin can be used.
+   *   (optional) The base tables on which this exposed form plugin can be used.
    *   If no base table is specified the plugin can be used with all tables.
    * @param bool $no_ui
-   *   Whether the plugin should be not selectable in the UI.
+   *   (optional) Whether the plugin should be not selectable in the UI.
    *   If it's set to TRUE, you can still use it via the API in config files.
+   *   Defaults to FALSE.
    * @param bool $register_theme
-   *   (optional) Whether or not to register a theme function automatically. Defaults to TRUE.
+   *   (optional) Whether to register a theme function automatically. Defaults
+   *   to TRUE.
    * @param class-string|null $deriver
    *   (optional) The deriver class.
    */

Did some docs improvements on commit.

Automatically closed - issue fixed for 2 weeks with no activity.