From 0b64903cb12cc342c3e0386ddc6224921c483581 Mon Sep 17 00:00:00 2001 From: vijay sawant Date: Wed, 22 Jan 2025 22:56:50 +0530 Subject: [PATCH] subscription page search field ui selector update (#1702) * subscription page search field ui selector update * implemented PF4 related comments * use SearchableViewMixinPF4 instead of SubscriptionSearchableViewMixin in class SearchableViewMixinPF4 (cherry picked from commit 4441e8d513e4fefcd45db351786bf51f6f89d134) --- airgun/views/subscription.py | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/airgun/views/subscription.py b/airgun/views/subscription.py index 14a38a3d0..34803eb19 100644 --- a/airgun/views/subscription.py +++ b/airgun/views/subscription.py @@ -10,27 +10,15 @@ from widgetastic_patternfly import BreadCrumb, Button from airgun.exceptions import ReadOnlyWidgetError -from airgun.views.common import BaseLoggedInView, SatTab, SearchableViewMixin +from airgun.views.common import BaseLoggedInView, SatTab, SearchableViewMixinPF4 from airgun.widgets import ( ConfirmationDialog, ItemsListReadOnly, ProgressBar, SatTable, - Search, ) -# Search field and button on Subscriptions page uses different locators, -# so subclass it and use it in our custom SearchableViewMixin -class SubscriptionSearch(Search): - search_field = TextInput(locator=(".//input[starts-with(@id, 'downshift-')]")) - search_button = Button('Search') - - -class SubscriptionSearchableViewMixin(SearchableViewMixin): - searchbox = SubscriptionSearch() - - class DeleteSubscriptionConfirmationDialog(ConfirmationDialog): confirm_dialog = Button('Delete') cancel_dialog = Button('Cancel') @@ -115,7 +103,7 @@ def fill(self, values): self.close() -class SubscriptionListView(BaseLoggedInView, SubscriptionSearchableViewMixin): +class SubscriptionListView(BaseLoggedInView, SearchableViewMixinPF4): """List of all subscriptions.""" table = SatSubscriptionsViewTable(